-
Notifications
You must be signed in to change notification settings - Fork 47
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
Switch to latest nvim-treesitter queries #36
Conversation
Mistakenly deleted it, this is a valid query.
Ah, mistakenly removed a highlight, apologies.
Understood, although there is 1 remaining color change left in this PR that is more intentional:
(although I think this changes rust-analyzer's decorations? LMK how you feel about that.) I was thinking these changes aren't controversial, but I could revert/put in different PR? Also feel free to edit this PR as you please as I won't be available until likely late tomorrow. Edit: previously, I changed the |
Seems more reasonable
Oh I didn't know lsp.type.struct affected that (rust-analyzer refuses to install on android so I couldn't check for breaking changes on Rust), reverted both changes then. Thanks for the patience. PR is just renames now. |
Was a new query.
Ah, I forgot to merge it, sorry. 2 weeks of testing, no issues, thank you! |
nvim-treesitter
recently made a bunch of breaking changes to their queries to stay up-to-date with treesitter/helix(nvim-treesitter/nvim-treesitter#5895), this PR should keep this colorscheme up-to-date.
For this PR, I mostly followed https://github.com/nvim-treesitter/nvim-treesitter/blob/master/CONTRIBUTING.md#highlights and took a peek at https://github.com/Iron-E/nvim-highlite/blob/master-v4/colors/highlite-ayu.lua
Let me know if you want me to just deprecate the old highlights instead of outright removing them.