Skip to content

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
Collaborator

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
Collaborator

not great not terrible :D

@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 gitui-org:master Mar 27, 2024
@extrawurst
Copy link
Collaborator

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 gitui-org#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