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 back to DerekStride/tree-sitter-sql #10

Open
notpeter opened this issue Jan 24, 2025 · 2 comments
Open

Switch back to DerekStride/tree-sitter-sql #10

notpeter opened this issue Jan 24, 2025 · 2 comments

Comments

@notpeter
Copy link
Contributor

Extension is currently using:

Which has seen little recent development. Consider switching back to upstream:

Not sure the original reason for using the fork.

@nervenes
Copy link
Collaborator

nervenes commented Jan 24, 2025

at the time i created this, dereks version was incompatible with wasm and i had to do some changes in the c code for it to work with zed. i dont know if that's the case any longer, but if it is you could just update the fork by pulling the upstream changes from derek 🤷‍♂

@notpeter
Copy link
Contributor Author

Thanks for the context @nervenes. I tried to build it with the current upstream main of tree-sitter-sql but it failed. I did only cursory investigation.

Also found this issue:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants