-
Notifications
You must be signed in to change notification settings - Fork 743
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
Add augeas lexer #1521
Add augeas lexer #1521
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@raphink Thanks for the submission :) I've left some comments below.
Thanks for the review @pyrmont. It'd be a good test to check the lexer against various lenses in https://github.com/hercules-team/augeas/tree/master/lenses (both curdir and tests/). Is there an easy way to do do that? |
@raphink Unfortunately, not really. Rouge comes with a command-line tool Does that help at all? |
Yes it does help, thanks! I've run it against a few files. These examples generate errors:
So essentially:
|
There, I've run |
@raphink Thanks for going through and trying out the lexer with those other files. Looks like you caught some tricky bugs! I've merged it into |
Great, thanks for your help @pyrmont |
This commit adds a lexer for Augeas. Co-authored-by: Michael Camilleri <mike@inqk.net>
This adds a lexer for Augeas lenses (https://augeas.net/)