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

update tree-sitter #54

Merged
merged 2 commits into from
Jan 21, 2025
Merged

update tree-sitter #54

merged 2 commits into from
Jan 21, 2025

Conversation

TomSkarabis
Copy link
Collaborator

Update Tree-Sitter dependency to the newest available version because of issues detailed in #53.

@TomSkarabis TomSkarabis requested a review from VonTum January 20, 2025 17:49
@TomSkarabis TomSkarabis self-assigned this Jan 20, 2025
@TomSkarabis
Copy link
Collaborator Author

@VonTum , can you run tree-sitter generate using this branch? I want to check if this is still dependent on the environment now (using the newest tree-sitter-cli version).

@TomSkarabis
Copy link
Collaborator Author

CI currently fails because the pipeline also fails upstream. I'm getting the same fails with and without this commit.

@VonTum
Copy link
Collaborator

VonTum commented Jan 21, 2025

@VonTum , can you run tree-sitter generate using this branch? I want to check if this is still dependent on the environment now (using the newest tree-sitter-cli version).

Sure, I get no git delta on:

lennart@Tumxedo:~/Desktop/sus-compiler/tree-sitter-sus$ tree-sitter --version
tree-sitter 0.24.7 (2660db97f9d5f9ff6f3af4c6a1c7fc9e5027ef8b)
lennart@Tumxedo:~/Desktop/sus-compiler/tree-sitter-sus$ ./tree.sh

@VonTum VonTum marked this pull request as ready for review January 21, 2025 13:51
@VonTum
Copy link
Collaborator

VonTum commented Jan 21, 2025

I reverted the last commit on master. I had forgotten to run ./test.sus_regression.sh

@VonTum VonTum merged commit 633432e into master Jan 21, 2025
1 check passed
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

Successfully merging this pull request may close these issues.

2 participants