avajs/eslint-plugin-ava

Skipping assertions is changing in AVA 1.0 #187

novemberborn posted onGitHub

https://github.com/avajs/ava/pull/1696 changes assertion skipping from t.skip.true() to t.true.skip().

From searching the rule documentation this impacts no-skip-assert and use-t-well.

Interestingly enough it looks as if use-t-well has some support for t.true.skip() even though it never worked.

Note that t.skip() no longer exists.


Note: This issue has a bounty, so it's expected that you are an experienced programmer and that you give it your best effort if you intend to tackle this. Don't forget, if applicable, to add tests, docs (double-check for typos). And don't be sloppy. Review your own diff multiple times and try to find ways to improve and simplify your code. Instead of asking too many questions, present solutions. The point of an issue bounty is to reduce my workload, not give me more. Include a 🦄 in your PR description to indicate that you've read this. Thanks for helping out 🙌 - @sindresorhus


@issuehunt has funded $60.00 to this issue.


posted by issuehunt-app[bot] over 5 years ago

@sindresorhus has rewarded $54.00 to @stroncium. See it on IssueHunt

  • :moneybag: Total deposit: $60.00
  • :tada: Repository reward(0%): $0.00
  • :wrench: Service fee(10%): $6.00
posted by issuehunt-app[bot] about 5 years ago

Fund this Issue

$60.00
Rewarded

Rewarded pull request

Recent activities

stroncium was rewarded by sindresorhus for avajs/eslint-plugin-ava# 187
about 5 years ago
stroncium submitted an output to  avajs/ eslint-plugin-ava# 187
over 5 years ago