-
Notifications
You must be signed in to change notification settings - Fork 738
WIP: Capture all comments in parser #6332
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
WIP: Capture all comments in parser #6332
Conversation
✅ Deploy Preview for nextflow-docs-staging ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Notes from today's meeting
We will investigate another approach in which the CST is converted directly into a formatting IR and used by the formatter.
We can also link CST nodes to AST nodes as needed to retrieve semantic information. But in the first iteration let's see how far we can get with just CST -> formatting IR -> formatter.
b7b4221 to
c1114bc
Compare
Uh oh!
There was an error while loading. Please reload this page.
This draft PR aims to add full support for finding comments in Nextflow source code and attaching them to the appropriate AST node. The additions to the parser are mirrored by the formatter classes used by the language server with the goal of addressing issues with the formatter removing comments.