Skip to content

Use HCL grammar from tree-sitter-grammars#25

Merged
MrSubidubi merged 2 commits into
zed-extensions:mainfrom
149segolte:patch-grammar
May 8, 2026
Merged

Use HCL grammar from tree-sitter-grammars#25
MrSubidubi merged 2 commits into
zed-extensions:mainfrom
149segolte:patch-grammar

Conversation

@149segolte
Copy link
Copy Markdown
Contributor

The grammar used is 2 years outdated at this point and the language has since added bunch of features that break it (eg: custom functions syntax).

Also, the upstream now redirects to tree-sitter-grammars/tree-sitter-hcl, the latest release there is v1.2.0 at commit fad991865fee927dd1de5e172fb3f08ac674d914.

@cla-bot
Copy link
Copy Markdown

cla-bot Bot commented May 7, 2026

We require contributors to sign our Contributor License Agreement, and we don't have @149segolte on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@149segolte
Copy link
Copy Markdown
Contributor Author

@cla-bot check

@cla-bot cla-bot Bot added the cla-signed label May 7, 2026
@cla-bot
Copy link
Copy Markdown

cla-bot Bot commented May 7, 2026

The cla-bot has been summoned, and re-checked this pull request!

@MrSubidubi MrSubidubi changed the title feat: move upstream grammar to latest location and tag Update grammar to the HCL grammar from tree-sitter-grammars May 8, 2026
@MrSubidubi MrSubidubi changed the title Update grammar to the HCL grammar from tree-sitter-grammars Use HCL grammar from tree-sitter-grammars May 8, 2026
Copy link
Copy Markdown
Contributor

@MrSubidubi MrSubidubi 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! And congratulations to your first contribution! 🎉

@MrSubidubi MrSubidubi merged commit 443c4b4 into zed-extensions:main May 8, 2026
5 checks passed
@149segolte 149segolte deleted the patch-grammar branch May 8, 2026 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants