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

feat(c): add text objects for statement conditions #576

Merged
merged 1 commit into from
Apr 3, 2024

Conversation

ribru17
Copy link
Member

@ribru17 ribru17 commented Mar 4, 2024

Allows @conditional.inner to include not just the body of e.g. an if statement, but also the condition itself (and conditions for while loops, do while, etc.)

@ilan-schemoul
Copy link
Contributor

You have committed to this project so you might have commit rights ?
If so you can probably merge it because I can confirm it's working and we need that very much

@ribru17
Copy link
Member Author

ribru17 commented Apr 1, 2024

I don't unfortunately, but I will try and bump this. Are you able to review @kiyoon?

@clason
Copy link
Collaborator

clason commented Apr 1, 2024

Tests are failing. (Cpp inherits from C, so the failure is meaningful.)

@ribru17
Copy link
Member Author

ribru17 commented Apr 1, 2024

Damn, thanks I'll check that out. And thanks for the invite! :D

@ilan-schemoul

This comment was marked as off-topic.

@ilan-schemoul

This comment was marked as duplicate.

@ribru17
Copy link
Member Author

ribru17 commented Apr 2, 2024

I think this PR is valid now but tests are failing due to some vlang issue. I will try to get a quick fix for that then rebase.

@clason clason merged commit 45f9096 into nvim-treesitter:master Apr 3, 2024
6 checks passed
@kiyoon
Copy link
Collaborator

kiyoon commented Apr 7, 2024

Thanks for your work. Btw I'm currently doing a short military service so I won't be able to review this month

@ribru17
Copy link
Member Author

ribru17 commented Apr 7, 2024

@kiyoon stay safe!

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.

4 participants