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
New: add option to report unused eslint-disable directives (fixes #9249) #9250
Conversation
bd013b8
to
0400514
Compare
LGTM |
@@ -32,26 +32,27 @@ function compareLocations(itemA, itemB) { | |||
* comment for two different rules is represented as two directives). |
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.
The business logic changes (i.e. the changes that actually handle directive comments rather than just passing options through the API) are all in this file.
0400514
to
3ce4d02
Compare
LGTM |
3ce4d02
to
a5e29e1
Compare
LGTM |
a5e29e1
to
8b60563
Compare
LGTM |
8b60563
to
0545dd0
Compare
LGTM |
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.
Whew, this was a lot, nice job @not-an-aardvark!
What is the purpose of this pull request? (put an "X" next to item)
[x] Add something to the core (see #9249)
What changes did you make? (Give an overview)
This adds the
--report-unused-disable-directives
CLI flag, and thereportUnusedDisableDirectives
option toCLIEngine
andLinter
, as described in #9249.Is there anything you'd like reviewers to focus on?
Nothing in particular