id-blacklist doesn't warn when renaming properties to blacklisted identifiers in destructuring assignment.
See original GitHub issueTell us about your environment
- ESLint Version: 7.0.0 (upcoming)
- Node Version: 10.15.3
- npm Version: 6.4.1
What parser (default, Babel-ESLint, etc.) are you using? default Please show your full configuration:
Configuration
id-blacklist: ["foo", "bar"] // Rule options
What did you do? Please include the actual source code causing the issue, as well as the command that you used to run ESLint.
const {foo: bar} = baz
What did you expect to happen?
ESLint should warn that bar
is a blacklisted identifier, but not foo
as it is a property from the original baz
object.
What actually happened? Please include the actual, raw output from ESLint. There is no warning for either identifier.
Context for this: A recent PR bugfix removed the errant warning on properties that were being renamed (foo
in the example above). In the course of this it ignores the following blacklisted bar
identifier. This is a follow up to https://github.com/eslint/eslint/pull/12792.
Are you willing to submit a pull request to fix this bug? Yes
Issue Analytics
- State:
- Created 4 years ago
- Comments:6 (6 by maintainers)
Top GitHub Comments
If there is no one working on, I will try. : )
I don’t think so!