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

Add a test_options for deno test #795

Closed
wants to merge 4 commits into from
Closed

Conversation

pipelight
Copy link

Make sure these boxes are checked before submitting your pull request:

  • Add fixtures and spec when implementing or updating a test runner
  • Update the README accordingly
  • Update the Vim documentation in doc/test.txt

It seems like it hasn't been implemented for every runner.

S/O to maintainers great tool!

@codeinabox
Copy link
Collaborator

Thank you for your PR. I've looked into this, and you are able to useg:test#javascript#denotest#options to set options, which I've validated with this test #798. Does that achieve what you need?

@codeinabox
Copy link
Collaborator

I've realised the existing documentation was a bit confusing, as it didn't make clear that test_options was specific to cargotest. I've got a PR #799 to update it, could you let me know if this clears things up?

@pipelight
Copy link
Author

Thank you very much for figuring this out.
My bad for submitting such a low quality PR, I was to much in a hurry to browse the documentation.
Great tool! Thanks again for maintaining it!

@pipelight pipelight closed this May 18, 2024
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.

None yet

2 participants