I'm new to writing tree-sitter parsers, so please correct me if this is an incorrect change.
All identifiers were being tagged as @variable.builtin. Looking at some of the documentation provided by nvim, @variable.builtin should be reserved for variables provided by the language (so this. in dart)
Changes
All (identifier)s are now tagged with @variable instead of @variable.builtin
(this) is now tagged as @variable.builtin
added a basic CI file so tests are run on new changes, happy to revert this if you'd prefer not having it in the repo
re-ran tree-sitter generate and tree-sitter build-wasm with the tree-sitter cli v0.20.9. This resulted in changes on master, leading me to think these were outdated from previous PRs
[!NOTE]
Running tree-sitter generate on v0.20.9, now results in the following output, these conflicts can probably be cleaned up in a later PR
TimWhiting
commented
9 months ago
I'm new to writing tree-sitter parsers, so please correct me if this is an incorrect change.
All identifiers were being tagged as
@variable.builtin. Looking at some of the documentation provided by nvim,@variable.builtinshould be reserved for variables provided by the language (sothis.in dart)Changes
(identifier)s are now tagged with@variableinstead of@variable.builtin(this)is now tagged as@variable.builtintree-sitter generateandtree-sitter build-wasmwith thetree-sittercliv0.20.9. This resulted in changes onmaster, leading me to think these were outdated from previous PRs