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

Bump to syntect 5.0.0 to e.g. start lazy-loading syntaxes #2181

Merged
merged 4 commits into from
May 7, 2022

Conversation

Enselic
Copy link
Collaborator

@Enselic Enselic commented May 4, 2022

This PR upgrades to syntect 5.0.0 (release notes).

Closes #915
Closes #951
Closes #1846
Closes #1854

Copy link
Owner

@sharkdp sharkdp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you very much. Looking forward to having this integrated!

src/printer.rs Outdated Show resolved Hide resolved
tests/syntax-tests/source/Vue/example.vue Outdated Show resolved Hide resolved
@Enselic Enselic changed the title Bump to syntect 5.0.0 to e.g. start lazy-loading themes Bump to syntect 5.0.0 to e.g. start lazy-loading syntaxes May 4, 2022
Enselic added a commit to Enselic/bat that referenced this pull request May 5, 2022
We do this to only have one invocation of `highlighter.highlight(...)`
so we don't need to change to `highlighter.highlight_line(...)` in two
places in sharkdp#2181.
Enselic added a commit to Enselic/bat that referenced this pull request May 5, 2022
We do this to only have one invocation of `highlighter.highlight(...)`
so we don't need to change to `highlighter.highlight_line(...)` in two
places in sharkdp#2181.
Enselic added a commit that referenced this pull request May 5, 2022
We do this to only have one invocation of `highlighter.highlight(...)`
so we don't need to change to `highlighter.highlight_line(...)` in two
places in #2181.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants