-
-
Notifications
You must be signed in to change notification settings - Fork 91
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 TOML highlighting support #217
Conversation
Thank you very much! Seems to work nicely! 👍️ |
Signed-off-by: Patrizio Bekerle <[email protected]>
25.2.3
There now is a new release, could you please test it and report if it works for you? |
Yep it works as intended! |
Great, thank you for testing! |
I noticed that I was mistakenly given credit for TOML support in the newest version release notes, when this was actually @LockeBirdsey who contributed. Is it possible to edit the typo? Thank you |
Oh! Thank you for mentioning! |
Signed-off-by: Patrizio Bekerle <[email protected]>
Tried my best to add TOML support. Everything is there except for multiline string handling, i.e
After some investigation and experimentation, it seems that when doing multiple
'''
or"""
while also switching between two block states, it will just fail silently (at least I couldn't find where it stopped trying to render). ThehighlightStringLiteral
function seems to clash but even disabling that inTOML
mode, I sill encountered this issue.