Tree-sitter rolling fixes, 1.129 edition #1300
Conversation
…where `color:red;` was interpreted as a tag and pseudoclass instead of a property-value pair.
…plus the names of abstract methods in class definitions.
DeeDeeG
left a comment
DeeDeeG
left a comment
There was a problem hiding this comment.
I trust this is right, I don't have a ton of context about the language features these changes are about, but... not in a mood to block it! Kind of a rubber-stamp, but I have read diffs and commit messages to know nominally what I'm approving, at least!
Glanced at the diff of tree-sitter-scss too, but that one is rather big!
|
Build (CI) fix should be in |
Yeah, lots of generated files with a Tree-sitter parser change. The changes to |
|
Nice, at least those are pretty compact changes. (I of course appreciate specs being added.) |
|
(the full change, for posterity) |
* Fix a bug in `tree-sitter-scss`… …where `color:red;` was interpreted as a tag and pseudoclass instead of a property-value pair. * [language-sass] Add grammar spec for fixed issue in `tree-sitter-scss` * [language-typescript] Highlight `abstract` modifier properly… …plus the names of abstract methods in class definitions.
We've finally got a Tree-sitter fix for the next version, whenever it may be!
Someone reported a bug against my
tree-sitter-scssparser, so I fixed it and updated the parser inlanguage-sass. I also added a grammar test for the issue. (More stuff should go in there, but I'm lazy now, and at least the infrastructure is present to add further grammar tests later.)