Skip to content

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

Open
ogaken-1 wants to merge 1 commit into
tyru:masterfrom
ogaken-1:fix/treesitter-hl_map-removed
Open

Fix treesitter based uncomment action doesn't work if has('nvim-0.8.0')#185
ogaken-1 wants to merge 1 commit into
tyru:masterfrom
ogaken-1:fix/treesitter-hl_map-removed

Conversation

@ogaken-1

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

Copy link
Copy Markdown

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
Copy Markdown
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

ogaken-1 commented Mar 29, 2023

Copy link
Copy Markdown
Author

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