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

Switch MATLAB tree-sitter parser #7388

Merged
merged 1 commit into from
Jun 22, 2023

Conversation

acristoffers
Copy link
Contributor

Product of #7386.

I believe I got the highlight right, or at least now I get a nice rainbow-vomit when using the doom_acario_dark theme. And no more quirks.

Textobjects and indents were tested and are inline with what I found in other grammars. That is to say "could you please take a look at the indents because I think it is broken, but it is defined the same as other languages so I don't know if it's broken or if it's just me who doesn't know how this editor works".

I simply copied the other queries and did not adapt them or anything. I not even know if helix supports them.

image

runtime/queries/matlab/highlights.scm Outdated Show resolved Hide resolved
runtime/queries/matlab/highlights.scm Outdated Show resolved Hide resolved
runtime/queries/matlab/injections.scm Outdated Show resolved Hide resolved
runtime/queries/matlab/locals.scm Show resolved Hide resolved
@the-mikedavis
Copy link
Member

If you run cargo xtask docgen and commit the results that will clear up the X in the docs CI

@acristoffers acristoffers force-pushed the master branch 3 times, most recently from 469bae6 to 8937396 Compare June 20, 2023 05:38
@the-mikedavis the-mikedavis added S-waiting-on-review Status: Awaiting review from a maintainer. A-language-support Area: Support for programming/text languages labels Jun 20, 2023
Copy link
Member

@the-mikedavis the-mikedavis 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!

@the-mikedavis the-mikedavis changed the title Changes MATLAB parser. Switch MATLAB tree-sitter parser Jun 20, 2023
@acristoffers
Copy link
Contributor Author

Do you want me to change the commit message too?

@the-mikedavis
Copy link
Member

It's not necessary: since it's a single commit we will probably squash+merge which will rename the commit and include the PR number

Copy link
Member

@pascalkuthe pascalkuthe left a comment

Choose a reason for hiding this comment

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

Thanks!

@pascalkuthe pascalkuthe merged commit 2c5288d into helix-editor:master Jun 22, 2023
@acristoffers
Copy link
Contributor Author

One question: do you update the tagged commit automatically or do I have to make pull-requests?

@pascalkuthe
Copy link
Member

It's done manually since grammar changes usually also require changes to the queries

wes-adams pushed a commit to wes-adams/helix that referenced this pull request Jul 4, 2023
mtoohey31 pushed a commit to mtoohey31/helix that referenced this pull request Jun 2, 2024
smortime pushed a commit to smortime/helix that referenced this pull request Jul 10, 2024
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 S-waiting-on-review Status: Awaiting review from a maintainer.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants