Skip to content

fix: depend on tree-sitter/go-tree-sitter#247

Merged
VladimirMakaev merged 1 commit intofwcd:mainfrom
hendrikvanantwerpen:fix-go-tree-sitter-dep
Mar 12, 2026
Merged

fix: depend on tree-sitter/go-tree-sitter#247
VladimirMakaev merged 1 commit intofwcd:mainfrom
hendrikvanantwerpen:fix-go-tree-sitter-dep

Conversation

@hendrikvanantwerpen
Copy link
Copy Markdown
Contributor

@hendrikvanantwerpen hendrikvanantwerpen commented Mar 11, 2026

Hi! We are using your grammar @endorlabs (🙇🏻 for maintaining this!) and ran into a tricky dependency problem. I tracked it down to this grammar's dependency on smacker/go-tree-sitter. This library has not been maintained since the Go bindings were adopted in the upstream tree-sitter org (link).

All upstream grammars have moved to tree-sitter/go-tree-sitter (example). I propose to do the same here to stay aligned with upstream changes and keep it as easy as possible to use multiple grammars in a single project.

@github-actions github-actions bot added the bindings Related to the Node.js/Rust/C bindings label Mar 11, 2026
@VladimirMakaev VladimirMakaev merged commit 9ca3ca9 into fwcd:main Mar 12, 2026
2 checks passed
@hendrikvanantwerpen hendrikvanantwerpen deleted the fix-go-tree-sitter-dep branch March 23, 2026 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bindings Related to the Node.js/Rust/C bindings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants