Skip to content
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

Preserve comments in AST #12

Merged
merged 2 commits into from
Feb 24, 2024
Merged

Conversation

jpihl
Copy link
Contributor

@jpihl jpihl commented Feb 21, 2024

Hi there,

I've been exploring tools for parsing proto files to extract documentation from messages and fields, and I found your cool project. I like the simplicity, but for my use case, I need to access the comments in the proto files as well - this PR solves that.
I don't know if this is something you think is within the scope of your project, but here goes.

All the best,

Jeppe

@criccomini
Copy link
Owner

Woah, this is great!! Definitely happy to add this functionality. Only concern is that we're modifying the Buf lex files. Just means we'll have to merge their changes in if they make any. Not a huge deal. Let me take a look at the PR and get back to you.

@criccomini criccomini merged commit d5613f1 into criccomini:main Feb 24, 2024
1 check passed
@criccomini
Copy link
Owner

Merged! Released in 1.1.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants