Skip to content

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
@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

2 participants