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 c3 language support to helix #11521

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

TheZombie1999
Copy link

No description provided.

Comment on lines 211 to 214
(integer_literal) @number
(real_literal) @number.float
(char_literal) @character
(bytes_literal) @number
Copy link
Member

Choose a reason for hiding this comment

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

Some of the scopes in this file need fixing https://docs.helix-editor.com/themes.html#syntax-highlighting

Copy link
Author

Choose a reason for hiding this comment

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

Is there a good way to debug these scopes ?

Copy link
Member

Choose a reason for hiding this comment

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

You can use the :tree-sitter-highlight-name command to show the capture name under the cursor

@the-mikedavis the-mikedavis added the A-language-support Area: Support for programming/text languages label Aug 19, 2024
@chrispaig3
Copy link

Looking forward to the C3 support!

Implementation of indent and textobjects querry for c3.
@theunixer
Copy link

I hope it will be added in the next release.

block-comment-tokens = { start = "/*", end = "*/" }
indent = { tab-width = 4, unit=" "}
language-servers = [ "c3-lsp" ]
grammar = "c3"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
grammar = "c3"

This key can be omitted since by default it matches the name of the language

@@ -417,6 +418,21 @@ grammar = "janet-simple"
name = "janet-simple"
source = { git = "https://github.com/sogaiu/tree-sitter-janet-simple", rev = "51271e260346878e1a1aa6c506ce6a797b7c25e2" }

[[grammar]]
name = "c3"
source = { git = "https://github.com/c3lang/tree-sitter-c3" , rev = "790a0326833cd647e00d8dec01268aa1ec2e3bdb" }
Copy link

Choose a reason for hiding this comment

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

Should update to the latest commit:
c6b1bae4d575f91e5c6c3c4af93bbec4e543ac5e

Copy link

Choose a reason for hiding this comment

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

Using this commit breaks the currently commited runtime queries

@sudokit
Copy link

sudokit commented Dec 2, 2024

Needs some updating. c3-treesitter repo commit hash is outdated and highlights.scm could also be checked that its up-to-date.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-language-support Area: Support for programming/text languages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants