-
Notifications
You must be signed in to change notification settings - Fork 83
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
Report comments via AST #417
Conversation
Furthermore, refine newline printing after comments and add more tests.
@teiesti, could you have a look if this suits your application? |
I am going to have a look. |
I was able to make the following work on my machine which looks very promising.
I can't wait to see this being stabilized. Thanks for the effort! Anyhow, I would like to see the pdoc documentation. Is there an easy way to generate it? @susuhahnml You may also want to look through this! |
There is not much to document though. It is just one additional type added to the AST. |
@teiesti, @susuhahnml, this should be ready. You can already try using the development packages on ubuntu/anaconda/pypi test. -R |
think about option to ignore comment ASTsThe snippet below shows some issues that should still (or not) be addressed before merging: