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

Add support for Rego syntax #918

Merged
merged 1 commit into from Jan 22, 2024
Merged

Conversation

jamietanna
Copy link
Contributor

@jamietanna jamietanna marked this pull request as ready for review January 22, 2024 15:53
@alecthomas alecthomas merged commit ae36e63 into alecthomas:master Jan 22, 2024
2 checks passed
@jamietanna jamietanna deleted the feature/rego branch January 22, 2024 21:08
@jamietanna
Copy link
Contributor Author

Argh sorry @alecthomas it looks like this hasn't actually worked OK - it looks like some of the text doesn't get recognised correctly.

2024-01-22-212019_1445x673_scrot

Any hints as what best to do here?

Happy to revert this PR until we've fixed it!

@alecthomas
Copy link
Owner

No worries

I'd add a test and iterate until it passes

@jamietanna
Copy link
Contributor Author

jamietanna commented Jan 22, 2024

Interesting, thanks - will give that a go tomorrow!

Will also update the README to add Rego 👍

@alecthomas
Copy link
Owner

FYI you can also use the chroma CLI to manually test your lexer: chroma --lexer <file.xml> <input>

@jamietanna
Copy link
Contributor Author

Thanks I had actually been doing that 🫣 but I changed themes to improve the output, but should have gone back to the default so I knew I'd not finished it off

@jamietanna jamietanna mentioned this pull request Jan 24, 2024
1 task
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.

None yet

2 participants