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

Add more operators #152

Merged
merged 7 commits into from
Feb 24, 2021
Merged

Add more operators #152

merged 7 commits into from
Feb 24, 2021

Conversation

icecream17
Copy link
Contributor

Adds more operators

Justification for operators is that += >> || are already there.

Moves -- && for consistency/visual purposes

(Ternary and keywords are not included as operators.)

jcs090218 and others added 5 commits February 6, 2021 13:08
...uh this is a lot. This might be kinda silly.

Ternary is an operator, but that's not included because of ":".
typeof, in, void, they're operators, but they're also keywords, so i'm not changing those either.
Update from main tree-sitter repo: Tolerate unescaped newlines in string literals (#140)
Copy link
Contributor

@maxbrunsfeld maxbrunsfeld left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@@ -1,6 +1,7 @@
name: build

on:
workflow_dispatch:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you remind me what this does?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, I was trying to test an addition to this pull request. It was on a pr of this... pull request. I added this while trying to do test the addition.
This allows for manual testing - but only if the default branch (master) has that too.

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.

3 participants