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 field names for common queries #94

Merged
merged 1 commit into from
Jan 4, 2022
Merged

Add field names for common queries #94

merged 1 commit into from
Jan 4, 2022

Conversation

alex-pinkus
Copy link
Owner

Some minor fixups from trying to use the parser, mainly adding fields
for the elements of most common nodes. Also some minor rearranging of
the AST: things like putting external parameter names within the
parameter nodes, rather than right before, which are non-backwards
compatible so it's best to do them early.

The corpus really didn't change much, but tree-sitter test -u
reformatted it. This is a one-time cost and now that we're using the
auto-updated formatting, the diffs will be much cleaner from now on.

Includes a version bump to 0.0.2, which is good because it's an
opportunity to republish the package with the publishing fixes from #93.

Some minor fixups from trying to use the parser, mainly adding fields
for the elements of most common nodes. Also some minor rearranging of
the AST: things like putting external parameter names within the
parameter nodes, rather than right before, which are non-backwards
compatible so it's best to do them early.

The corpus really didn't change much, but `tree-sitter test -u`
reformatted it. This is a one-time cost and now that we're using the
auto-updated formatting, the diffs will be much cleaner from now on.

Includes a version bump to `0.0.2`, which is good because it's an
opportunity to republish the package with the publishing fixes from #93.
@alex-pinkus alex-pinkus merged commit b188a7a into main Jan 4, 2022
@alex-pinkus alex-pinkus deleted the declare-fields branch January 4, 2022 03:05
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.

1 participant