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

Tree-sitter rolling fixes, 1.124 edition #1148

Merged

Conversation

savetheclocktower
Copy link
Contributor

This barely needs to exist at this point, but I wanted to make sure we shipped something in 1.124.

Kidding! Mostly kidding!

…and emit an event in `IndentResolver::suggestedIndentForBufferRow` that was missed last time.
(Should've made it into last month’s!)
I _think_ this is happening because we're not waiting for the watcher to start. `nsfw` goes async when the watcher is created _and_ when it's started.
Copy link
Member

@confused-Techie confused-Techie left a comment

Choose a reason for hiding this comment

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

This one looks much like the others. Love seeing all the docs included! Lets get it merged

@savetheclocktower savetheclocktower merged commit aee35aa into pulsar-edit:master Dec 16, 2024
102 of 103 checks passed
@savetheclocktower savetheclocktower deleted the tree-sitter-december branch December 16, 2024 00:35
savetheclocktower added a commit that referenced this pull request Dec 27, 2024
* Add comments to `WASMTreeSitterLanguageMode`…

…and emit an event in `IndentResolver::suggestedIndentForBufferRow` that was missed last time.

* [language-c] Fix highlighting of `&` in a reference declarator

(Should've made it into last month’s!)

* [language-c] Move `reference_declarator` declaration to C++-only file…

…because it's not valid C.

* Attempt to fix chronically failing spec

I _think_ this is happening because we're not waiting for the watcher to start. `nsfw` goes async when the watcher is created _and_ when it's started.
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.

2 participants