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

syntect additional file type support #2160

Merged

Conversation

martihomssoler
Copy link
Contributor

@martihomssoler martihomssoler commented Mar 26, 2024

This Pull Request fixes/closes #2005.

It changes the following:

  • add two-face as dependency to extend syntect supported file types
  • add two-face feature flags alongside syntect feature flags
  • change default SYNTAX_SET to two_face's default (still no newlines)

I followed the checklist:

  • [] I added unittests (not needed)
  • I ran make check without errors
  • I tested the overall application
  • I added an appropriate item to the changelog

@extrawurst
Copy link
Owner

can you run a quick comparison in generated binary size by this? both in release and debug

@martihomssoler
Copy link
Contributor Author

master branch (without two-face)

debug -> 17.9 MB
release -> 5.8 MB

syntect branch (with two-face)

debug -> 18.2 MB
release -> 6.3 MB

@extrawurst
Copy link
Owner

not great not terrible :D

CHANGELOG.md Outdated Show resolved Hide resolved
@extrawurst extrawurst force-pushed the 2005-syntect-additional-file branch from 859729b to 7172bda Compare March 27, 2024 13:25
@extrawurst extrawurst merged commit 3b7b443 into extrawurst:master Mar 27, 2024
18 checks passed
@extrawurst
Copy link
Owner

Thank you!

@martihomssoler martihomssoler deleted the 2005-syntect-additional-file branch April 6, 2024 19:02
IndianBoy42 pushed a commit to IndianBoy42/gitui that referenced this pull request Jun 4, 2024
closes extrawurst#2005

Co-authored-by: MHS <mhs@histolution.com>
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.

Syntax highlighting: Typescript files not highlighted
2 participants