Skip to content
This repository has been archived by the owner on Aug 7, 2023. It is now read-only.

Support new reportUnusedDisableDirectives option #1130

Open
ehmicky opened this issue Apr 17, 2018 · 2 comments
Open

Support new reportUnusedDisableDirectives option #1130

ehmicky opened this issue Apr 17, 2018 · 2 comments

Comments

@ehmicky
Copy link

ehmicky commented Apr 17, 2018

Issue Type

Feature Request

Issue Description

I would like to run ESLint in Atom with specific CLI flags. Unless I haven't checked it propertly, this is not possible with linter-eslint.

The problem arose for me with ESLint 5 which conflicts with the rule eslint-comments/no-unused-disabled. This rule is to be deprecated and replaced with the CLI flag --report-unused-disable-directives.

Another option would be to add a configuration property for --report-unused-disabled-directives.

@Arcanemagus
Copy link
Member

We are using the Node.js API, not the CLI interface so it's impossible to do this exactly as requested unfortunately.

For your specific flag it looks like there is a new CLIEngine option available that controls this behavior: reportUnusedDisableDirectives

I'll go ahead and change the title to reflect that as the goal of this 😉.

@Arcanemagus Arcanemagus changed the title Allow passing specific CLI flags to ESLint Support new reportUnusedDisableDirectives option Apr 17, 2018
@ehmicky
Copy link
Author

ehmicky commented Apr 17, 2018

Yes sorry, that makes sense. Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants