feat: migrate to nvim-treesitter 'main'#34
Conversation
b631c61 to
22b9006
Compare
|
Hey thank you! About time for me to upgrade my treesitter install as well 😁 |
joshzcold
left a comment
There was a problem hiding this comment.
Works well on treesitter master and I think thats what really counts for most users right now. Thank you for making sure the main branch doesn't just crash for the remainder of the users.
Can't really get :Python treesitter wrap_cursor working without a visual selection.
vim.treesitter.get_node() is turning up blank on the python parser on the main branch.
Does it work for you?
Enter a file, go to a in the file and try the command.
Definately not an issue with your PR just curious.
Looks like I need to fix something on the tests thats just related to Mini.test and doesn't effect your PR.
|
In fact nothing comes up in python on treesitter
|
|
Nevermind, I am running into nvim-treesitter/nvim-treesitter#8010 Was able to get functionaltiy working when working through that issue. I am going to merge this and fix tests after the fact. |
|
@Quitlox I reverted the Tests are passing on my machine and github actions. |
|
Thanks for merging 😄 As for the tests, the problem might be that See: https://github.com/nvim-treesitter/nvim-treesitter/blob/main/doc/nvim-treesitter.txt#L112 |
This PR migrates the plugin to use the
vim.treesitter.*API, as the APIs fromnvim-treesitterhave been removed in themainbranch. This should be compatible with both themasterandmainversion ofnvim-treesitter(except forminimal_init.lua, which assumes themainversion).Copilot was used to make these changes, with manual cleanup, testing and review.