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 a -lang option for setting the highlighting language #180

Merged
merged 12 commits into from
Jan 2, 2024

Conversation

walles
Copy link
Owner

@walles walles commented Jan 2, 2024

Works only for files.

Fixes #178.

This changes the highlighting API to take a string instead of a file
because strings are what Chroma can highlight.

With this change in place, we should invoke the highlighter after
we're done reading the input. This will allow highlighting both files
(as before) and streaming input.

This also moves the responsibility of checking the file against the max
highlight size from the highlighter to the caller.
After testing. The invalid value is already printed by the framework, no
need to include it in the error message.
@walles walles changed the title johan/force format Add a -lang option for setting the highlighting language Jan 2, 2024
@walles walles merged commit 83c1c7b into master Jan 2, 2024
2 checks passed
@walles walles deleted the johan/force-format branch January 2, 2024 15:18
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.

filetype colorization
1 participant