New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat!: no-unused-vars
default caughtErrors to 'all'
#18043
feat!: no-unused-vars
default caughtErrors to 'all'
#18043
Conversation
✅ Deploy Preview for docs-eslint ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Could you change the docs too? |
D'oh, right, thanks. |
We need to mention this in the migration guide, where we have mentioned other breaking changes as well. |
@@ -33,6 +33,7 @@ The lists below are ordered roughly by the number of users each change is expect | |||
* [`no-restricted-imports` now accepts multiple config entries with the same `name`](#no-restricted-imports) | |||
* [`"eslint:recommended"` and `"eslint:all"` strings no longer accepted in flat config](#string-config) | |||
* [`no-inner-declarations` has a new default behavior with a new option](#no-inner-declarations) | |||
* [`no-unused-vars` now defaults `caughtErrors` to `"all"`](#no-unused-vars) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As an aside, it would be nice in a separate PR to group all default rule option changes together in this list or a sub-list.
Co-authored-by: Milos Djermanovic <milos.djermanovic@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM pending @amareshsm's grammar update.
Co-authored-by: Amaresh S M <amareshsm13@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Prerequisites checklist
What is the purpose of this pull request? (put an "X" next to an item)
[ ] Documentation update
[ ] Bug fix (template)
[ ] New rule (template)
[x] Changes an existing rule (template)
[ ] Add autofix to a rule
[ ] Add a CLI option
[ ] Add something to the core
[ ] Other, please explain:
What changes did you make? (Give an overview)
Changes the default value for the
no-unused-vars
rule'scaughtErrors
option from"none"
to"all"
.Fixes #17974.
Is there anything you'd like reviewers to focus on?