search

eslint-community / eslint-plugin-promise

Enforce best practices for JavaScript promises
ISC License
939 stars 91 forks source link

feat: add `allowThenStrict` option #522

Open brettz9 opened 1 month ago

brettz9 commented 1 month ago

What is the purpose of this pull request?

What changes did you make? (Give an overview)

Added an allowThenStrict option which will fail on then(fn1, fn2) but pass on then(null, fn)

Also cleaned up the allowThen docs and tests.

Closes #52

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (bbcfcbf) to head (54151f4). Report is 39 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #522 +/- ## ========================================= Coverage 100.00% 100.00% ========================================= Files 25 26 +1 Lines 649 699 +50 Branches 250 268 +18 ========================================= + Hits 649 699 +50 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.