-
-
Notifications
You must be signed in to change notification settings - Fork 804
Open
Labels
A-LinterArea: linterArea: linterL-JavaScriptLanguage: JavaScript and super languagesLanguage: JavaScript and super languagesS-Help-wantedStatus: you're familiar with the code base and want to help the projectStatus: you're familiar with the code base and want to help the projectgood first issueGood for newcomersGood for newcomers
Description
Description
Rule Source: https://eslint.org/docs/latest/rules/preserve-caught-error
This is a new rule introduced in eslint v9.35.0.
Name suggestions:
- noRethrowWithoutCause
- useCauseInRethrow
- useCaughtError
IMO, It should not be recommended by default for the reasons listed in the "When Not To Use It" section in the rule source.
It would be nice if we could figure out why the requireCatchParameter option is false by default. Seems a bit counter productive to me.
Want to contribute? Let us know you are interested! We will assign you to the issue to prevent several people to work on the same issue. Don't worry, we can unassign you later if you are no longer interested in the issue! Read our contributing guide and analyzer contributing guide.
danielfsousa, romain-blatrix and kedevked
Metadata
Metadata
Assignees
Labels
A-LinterArea: linterArea: linterL-JavaScriptLanguage: JavaScript and super languagesLanguage: JavaScript and super languagesS-Help-wantedStatus: you're familiar with the code base and want to help the projectStatus: you're familiar with the code base and want to help the projectgood first issueGood for newcomersGood for newcomers