eslint-plugin-promise

eslint-plugin-promise copied to clipboard

chore: use 'files' instead of .npmignore

2

**What is the purpose of this pull request?** - [ ] Documentation update - [ ] Bug fix - [ ] New rule - [ ] Changes an existing rule...

midgleyc

feat(rule): add `wrap-await-with-try-catch` rule

2

**What is the purpose of this pull request?** - [ ] Documentation update - [ ] Bug fix - [x] New rule - [ ] Changes an existing rule -...

can-keklik

no-callback-in-promise: call should be allowed if promise does not return

2

### Description As a general rule, callbacks should not be called inside promises and, as a general rule, it is advisable to "nodeify" the promise call to properly work within...

gabmontes

Prefer calling `then` and `catch` separately to combining them into a single `then`

4

Essentially, `Promise.prototype.then` allows taking in two arguments to respond to both resolved and rejected promises. Essentially, `p.then(f, g)` is equivalent to `p.catch(g).then(f)`; note that the catch comes first since errors...

ghost

False-positive for "error" causing issues in prefer-await-to-callbacks rule

11

see: https://github.com/xjamundx/eslint-plugin-promise/pull/202#issuecomment-777651001 The way that this rule detects a callback function is anything with a function signature of: ```js function(err) { // or now "error" // you look like a...

xjamundx

Unable to report unhandled promises

3

### Description This, I guess, is more of a feature request than an issue: It would be great to be able to report all the un-handled promises. Meaning: for every...

jazzfog

### Description It's a feature that will help in keeping track of contributions made by the contributors and list them in the readme. This all will be handled by [all-contributors](https://allcontributors.org/)...

iamdarshshah

In some places we recommended using a 3rd party lib, but this stuff is built-in to most environments now.

xjamundx

I would like to suggest a rule to prevent calling the promise `catch` method without any argument. It works "differently" than the `catch` block and it might lead to bugs...

sonnyp

catch must re-throw

13

I've heard that `catch()` should re-throw the error or `return Promise.reject(err)` anyway familiar with this? It's so you don't break the promise chain.

xjamundx