-
Notifications
You must be signed in to change notification settings - Fork 87
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
Support extra_args #106
base: main
Are you sure you want to change the base?
Support extra_args #106
Conversation
23f25e4
to
693f8c1
Compare
This will indeed be helpful |
@izifortune FYI, I've started a new |
@resurtm Thanks and thanks for the suggestion. I personally consider this project somewhat unmaintained due to the lack of activity so I doubt I could get in contact with the maintainer although I haven't tried. I know the author of neotest (@rcarriga) has write access as a collaborator so he could merge stuff as well or perhaps give me access. On top of that, I'm already a collaborator of one neotest adapter and the maintainer of another so I'm not sure I would have time to maintain this repo unless your suggestion is to get write access just for my two open PRs. I can see the maintainer has an email so I'll try writing. |
This PR adds support for
neotest.RunArgs.extra_args
passed tobuild_spec
for adding additional arguments to the test process vianeotest.run.run
.An example use case is running tests and updating snapshots:
Other changes
is_test_file
tests that weren't run asynchronously but was calling async functions.