Skip to content
This repository has been archived by the owner on May 27, 2024. It is now read-only.

Fix: capture multiple group with same name (Alternative to https://github.com/neovim/neovim/pull/17099) #48

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

binhtran432k
Copy link

@binhtran432k binhtran432k commented Jun 12, 2023

This PR will add a workaround for capture multiple group with same name (Alternative to neovim/neovim#17099). The solution is add index to capture and get only the name in strategies

I only fixed html query first. @HiPhish, you should solve the remain queries by yourself. Thanks for a nice plugin.

This is a demo:
image

@HiPhish
Copy link
Owner

HiPhish commented Jun 12, 2023

That's a really clever hack, I love it. But it is still a hack and I prefer to avoid hacks. I think energy would be better spent solving the upstream issue in Neovim so everyone can profit. On the other hand, this could be a stopgap measure which I can leave undocumented and then silently remove again once Neovim is fixed. I think I'll have to think about this for a while.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants