-
Notifications
You must be signed in to change notification settings - Fork 60
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support new godoc spec enforced by gofmt #82
Conversation
This is temporary so that we can go run from this repo waiting for the PR princjef#82 to be reviewed and merge
This is temporary so that we can go run from this repo waiting for the PR princjef#82 to be reviewed and merge
e9b2352
to
01a1de8
Compare
This is temporary so that we can go run from this repo waiting for the PR princjef#82 to be reviewed and merge
Make use of go/doc and go/doc/comment std packages to parse and ender godoc paragraph
01a1de8
to
aa089f4
Compare
This is temporary so that we can go run from this repo waiting for the PR princjef#82 to be reviewed and merge
9a3fc53
to
7d0517e
Compare
This is temporary so that we can go run from this repo waiting for the PR princjef#82 to be reviewed and merge
762bc79
to
5937c86
Compare
This is temporary so that we can go run from this repo waiting for the PR princjef#82 to be reviewed and merge
@phsym thanks for the PR! This is a big update that clearly took some time to put together. I unfortunately already had most of the implementation put together in a separate change which I ended up using for the final version of the go 1.19 support, so I won't be able to merge this as is. It looks like you may have some specific functionality (such as the GOPRIVATE support) which I didn't implement in my version. If you're interested in trying to port that and other relevant logic over I'd happily work with you to get it merged, though I understand it's been a while and you may not have the time/inclination at this point. Let me know your thoughts. |
Hi ! Thanks for this awesome tool.
As stated in #81 unfortunately, it does not play well with the new godoc spec. Like headers, or links references.
This PR is an attempt to delegate the parsing of docstrings to
go/doc/comment.Parser
, and markdown rendering (escaping) togo/doc/comment.Printer
as suggested in a comment of #81I may have missed things but from the tests I did on a large codebase, it seems to be working quite well.
I also updated to go 1.19
You can give it a try with
Fixes #81