-
Notifications
You must be signed in to change notification settings - Fork 2
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
Update + Add Syntaxes #3
base: master
Are you sure you want to change the base?
Conversation
Fix build script (i.e. revert to before update).
Hm, possibly something not quite right in this repo. Builds fine with a local link against the reason binds, but trying to build
Need to actually understand that and work out if its an API change (and if so, where to change that) or if its a build issue. EDIT: Ah! Everyone of the new grammars includes a |
These are needed for these more complex parsers.
Looks like this might need to be a bit more involved actually, since we currently build with |
Its just a symlink and it is causing issues with the Windows build.
We don't use it, and it is causing issues. I've added a file in place to explain this.
This bumps us up to the latest master of
tree-sitter
.I've also added CPP, JS, TS, TSX and Python parsers. I can add more later, but wanted to check these were actually added correctly.
Steps taken:
build-lib
script to bring back our changes.npm install
and copy over theparser.c
.build-lib
build step andar
step.Still WIP as I've not tested with the upstream reason binds yet.