1362972 - Add an ESLint rule to detect some cases of promise chains that are not terminated

Status: NEW

Description

[:Paolo Amadini]

Most promise chains should end with ".catch(Cu.reportError);" or ".catch(() => {});".

We cannot detect all misuses, but we can likely detect the most common ones by checking that return value of ".then()" or a call to an async function is never ignored, but is stored in a variable, propagated via "return", or fed to ".catch()".

Comment 1

[Mark Banner (:standard8)]

We could potentially use some or all of the rules from this plugin: https://github.com/xjamundx/eslint-plugin-promise