Commit e6b5232
tools: enable no-useless-catch lint rule
This commit enables ESLint's no-useless-catch rule.
PR-URL: #25236
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: James M Snell <jasnell@gmail.com>1 parent f944a75 commit e6b5232
2 files changed
Lines changed: 2 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -234,6 +234,7 @@ module.exports = { | |||
| 234 | 234 | variables: false, | |
| 235 | 235 | }], | |
| 236 | 236 | 'no-useless-call': 'error', | |
| 237 | + 'no-useless-catch': 'error', | ||
| 237 | 238 | 'no-useless-concat': 'error', | |
| 238 | 239 | 'no-useless-constructor': 'error', | |
| 239 | 240 | 'no-useless-escape': 'error', | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -80,7 +80,7 @@ class Script extends ContextifyScript { | |||
| 80 | 80 | // Calling `ReThrow()` on a native TryCatch does not generate a new | |
| 81 | 81 | // abort-on-uncaught-exception check. A dummy try/catch in JS land | |
| 82 | 82 | // protects against that. | |
| 83 | - try { | ||
| 83 | + try { // eslint-disable-line no-useless-catch | ||
| 84 | 84 | super(code, | |
| 85 | 85 | filename, | |
| 86 | 86 | lineOffset, | |
0 commit comments