Skip to content
This repository has been archived by the owner on Dec 12, 2024. It is now read-only.

Implement operator precedence. #22

Merged
merged 1 commit into from
Dec 5, 2024

Conversation

hugopl
Copy link
Contributor

@hugopl hugopl commented Dec 5, 2024

I just took a look on how tree-sitter-c parser does it.

This also adds some missing operators.

Parenthesis still not implemented.

@Axlefublr
Copy link

seeing all these PRs from you makes me so happy 🥹

lack of working treesitter is the only reason I'm not using crystal currently, so keep up the good work! I appreciate you, and you @nobodywasishere for accepting the PRs so quickly!

@hugopl
Copy link
Contributor Author

hugopl commented Dec 5, 2024

This probably would need more tests, but write tests for tree-sitter is very easy and we can do later... or when a bug appears 😅

I just took a look on how tree-sitter-c parser does it.

This also adds some missing operators.

Parenthesis still not implemented.
@hugopl
Copy link
Contributor Author

hugopl commented Dec 5, 2024

I forgot to update the tests.. now it will be green.

@nobodywasishere nobodywasishere merged commit ca96eb8 into crystal-lang-tools:main Dec 5, 2024
3 checks passed
@hugopl hugopl deleted the expressions branch December 5, 2024 18:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants