-
-
Notifications
You must be signed in to change notification settings - Fork 240
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(neotest): add watch keybindings #1185
feat(neotest): add watch keybindings #1185
Conversation
Review ChecklistDoes this PR follow the [Contribution Guidelines](development guidelines)? Following is a partial checklist: Proper conventional commit scoping:
|
e9a1779
to
8b94474
Compare
8b94474
to
eb2b1cd
Compare
I'm thinking to refactor this after the PR approval, in order to follow the standard process to map keybingings used in here |
eb2b1cd
to
303dae8
Compare
Refactor first please. :) |
eb70b12
to
b2f8ed1
Compare
b2f8ed1
to
5acebee
Compare
📑 Description
Add mappings to simplify the usage of watch-related commands when using
neotest
.It also refactor the keybindings to follow current standards using an
opts
function instead of a table.ℹ Additional Information
The API for the usage of
neotest.watch.stop()
is not well defined in terms of the type information for the usage.This creates the necessity to use
@diagnostic disable-next-line: missing-parameter
.For a closer look, see https://github.com/nvim-neotest/neotest/blob/master/doc/neotest.txt#L626