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

Fix treesitter based uncomment action doesn't work if has('nvim-0.8.0') #185

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ogaken-1
Copy link

@ogaken-1 ogaken-1 commented Jan 26, 2023

Fix #183

Use vim.treesitter.get_captures_at_pos(bufnr, row, col) instead.

dshoreman added a commit to dshoreman/caw.vim that referenced this pull request Feb 2, 2023
Fixes an issue where trying to uncomment will result in "attempt to
index field 'hl_map' (a nil value)..." when used with treesitter.

Based on the code in [this comment][1] and tyru#185, but tweaked
slightly to avoid messing with indentation of the existing code.

[1]: Shougo/dein.vim#495 (comment)
@ogaken-1
Copy link
Author

@tyru
I am glad to see that you have resumed your activities.

Regarding the broken integration with nvim-treesitter, I apologise for creating something that conflicts with other PRs.

I have no particular objection to closing this, so please offer to do so.

@ogaken-1 ogaken-1 force-pushed the fix/treesitter-hl_map-removed branch from d93b3bb to cf9d7ad Compare March 29, 2023 03:18
@ogaken-1
Copy link
Author

ogaken-1 commented Mar 29, 2023

I think the issue has not solved now.

image

I rebased this pr for master.

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.

caw.vim can't works well with treesitter
1 participant