-
Notifications
You must be signed in to change notification settings - Fork 115
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
Prebuild through GitHub Actions #140
Conversation
82513e2
to
4776983
Compare
.github/workflows/build.yml
Outdated
- 12 | ||
- 14 | ||
- 16 | ||
- 18 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have to work our way up to Node 18/20. node-tree-sitter has superstring in its devDependencies
Line 26 in 7f67305
"superstring": "^2.4.2", |
superstring doesn't support Node 17+, so if you try to run
git clone https://github.com/tree-sitter/node-tree-sitter.git
cd node-tree-sitter
npm install
with Node 18 or 20, it will fail.
283d35a
to
0d724ea
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!!
# - 18 | ||
# - 20 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add a comment with an issue number to help readers understand what is blocking these versions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure. I was planning on merging #141 before this one, just waiting on someone (Max) to double check it, but I don't have to.
18c9e0e
to
ebeff3f
Compare
ebeff3f
to
6d959a2
Compare
just to test that it's possible for now. Also update the dependencies
Closes #126