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

Extend groovy support #9677

Merged
merged 5 commits into from
Mar 17, 2024
Merged

Conversation

britter
Copy link
Contributor

@britter britter commented Feb 20, 2024

runtime/queries/groovy/locals.scm Outdated Show resolved Hide resolved
runtime/queries/groovy/indents.scm Outdated Show resolved Hide resolved
@kirawi kirawi added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. A-language-support Area: Support for programming/text languages labels Feb 21, 2024
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.

The queries look good but you will need to merge master / rebase and resolve the conflict and also run cargo xtask docgen and commit the changes

britter and others added 5 commits February 28, 2024 13:29
Use more complete parser introduced in nvm-treesitter in
nvim-treesitter/nvim-treesitter@d4dac52
Co-authored-by: Michael Davis <mcarsondavis@gmail.com>
It was copied from the tree-sitter repository but is not
compatiblw with the way indent queries are implemented
in Helix.
@britter
Copy link
Contributor Author

britter commented Feb 28, 2024

@the-mikedavis sorry for the additional turn around. All should be good now.

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.

Thanks!

@the-mikedavis the-mikedavis added S-waiting-on-review Status: Awaiting review from a maintainer. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Feb 28, 2024
@britter
Copy link
Contributor Author

britter commented Mar 12, 2024

@the-mikedavis who can we ask for a second review to get this merged?

@britter
Copy link
Contributor Author

britter commented Mar 16, 2024

@archseer do you have some time to check this? I'd really like to get this merged 💚

@pascalkuthe pascalkuthe merged commit f7913c1 into helix-editor:master Mar 17, 2024
6 checks passed
@britter britter deleted the britter/groovy-gradle branch March 18, 2024 19:01
Desdaemon pushed a commit to Desdaemon/helix that referenced this pull request Mar 26, 2024
* Extend groovy support

Use more complete parser introduced in nvm-treesitter in
nvim-treesitter/nvim-treesitter@d4dac52

* Update runtime/queries/groovy/locals.scm

Co-authored-by: Michael Davis <mcarsondavis@gmail.com>

* Drop indent.scm for groovy

It was copied from the tree-sitter repository but is not
compatiblw with the way indent queries are implemented
in Helix.

* Adapt groovy highlights to helix syntax

* Update documentation

---------

Co-authored-by: Michael Davis <mcarsondavis@gmail.com>
postsolar pushed a commit to postsolar/helix that referenced this pull request Apr 4, 2024
* Extend groovy support

Use more complete parser introduced in nvm-treesitter in
nvim-treesitter/nvim-treesitter@d4dac52

* Update runtime/queries/groovy/locals.scm

Co-authored-by: Michael Davis <mcarsondavis@gmail.com>

* Drop indent.scm for groovy

It was copied from the tree-sitter repository but is not
compatiblw with the way indent queries are implemented
in Helix.

* Adapt groovy highlights to helix syntax

* Update documentation

---------

Co-authored-by: Michael Davis <mcarsondavis@gmail.com>
postsolar pushed a commit to postsolar/helix that referenced this pull request Apr 4, 2024
* Extend groovy support

Use more complete parser introduced in nvm-treesitter in
nvim-treesitter/nvim-treesitter@d4dac52

* Update runtime/queries/groovy/locals.scm

Co-authored-by: Michael Davis <mcarsondavis@gmail.com>

* Drop indent.scm for groovy

It was copied from the tree-sitter repository but is not
compatiblw with the way indent queries are implemented
in Helix.

* Adapt groovy highlights to helix syntax

* Update documentation

---------

Co-authored-by: Michael Davis <mcarsondavis@gmail.com>
postsolar pushed a commit to postsolar/helix that referenced this pull request Apr 4, 2024
* Extend groovy support

Use more complete parser introduced in nvm-treesitter in
nvim-treesitter/nvim-treesitter@d4dac52

* Update runtime/queries/groovy/locals.scm

Co-authored-by: Michael Davis <mcarsondavis@gmail.com>

* Drop indent.scm for groovy

It was copied from the tree-sitter repository but is not
compatiblw with the way indent queries are implemented
in Helix.

* Adapt groovy highlights to helix syntax

* Update documentation

---------

Co-authored-by: Michael Davis <mcarsondavis@gmail.com>
postsolar pushed a commit to postsolar/helix that referenced this pull request Apr 20, 2024
* Extend groovy support

Use more complete parser introduced in nvm-treesitter in
nvim-treesitter/nvim-treesitter@d4dac52

* Update runtime/queries/groovy/locals.scm

Co-authored-by: Michael Davis <mcarsondavis@gmail.com>

* Drop indent.scm for groovy

It was copied from the tree-sitter repository but is not
compatiblw with the way indent queries are implemented
in Helix.

* Adapt groovy highlights to helix syntax

* Update documentation

---------

Co-authored-by: Michael Davis <mcarsondavis@gmail.com>
mtoohey31 pushed a commit to mtoohey31/helix that referenced this pull request Jun 2, 2024
* Extend groovy support

Use more complete parser introduced in nvm-treesitter in
nvim-treesitter/nvim-treesitter@d4dac52

* Update runtime/queries/groovy/locals.scm

Co-authored-by: Michael Davis <mcarsondavis@gmail.com>

* Drop indent.scm for groovy

It was copied from the tree-sitter repository but is not
compatiblw with the way indent queries are implemented
in Helix.

* Adapt groovy highlights to helix syntax

* Update documentation

---------

Co-authored-by: Michael Davis <mcarsondavis@gmail.com>
Vulpesx pushed a commit to Vulpesx/helix that referenced this pull request Jun 7, 2024
* Extend groovy support

Use more complete parser introduced in nvm-treesitter in
nvim-treesitter/nvim-treesitter@d4dac52

* Update runtime/queries/groovy/locals.scm

Co-authored-by: Michael Davis <mcarsondavis@gmail.com>

* Drop indent.scm for groovy

It was copied from the tree-sitter repository but is not
compatiblw with the way indent queries are implemented
in Helix.

* Adapt groovy highlights to helix syntax

* Update documentation

---------

Co-authored-by: Michael Davis <mcarsondavis@gmail.com>
smortime pushed a commit to smortime/helix that referenced this pull request Jul 10, 2024
* Extend groovy support

Use more complete parser introduced in nvm-treesitter in
nvim-treesitter/nvim-treesitter@d4dac52

* Update runtime/queries/groovy/locals.scm

Co-authored-by: Michael Davis <mcarsondavis@gmail.com>

* Drop indent.scm for groovy

It was copied from the tree-sitter repository but is not
compatiblw with the way indent queries are implemented
in Helix.

* Adapt groovy highlights to helix syntax

* Update documentation

---------

Co-authored-by: Michael Davis <mcarsondavis@gmail.com>
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.

4 participants