Add no-unreadable-array-destructuring rule
#199
Conversation
There was a problem hiding this comment.
It would be nice to have additional tests for the array destructuring in arguments, like function a([, b]) {}. From what I have gathered, it should handle this case well, but more test cases can't hurt.
Thanks @janowsiany for working on this!
jfmengels
left a comment
jfmengels
left a comment
There was a problem hiding this comment.
LGTM, thanks @janowsiany!
sindresorhus
changed the title
no-unreadable-array-destructuring rule
|
Thanks for working on this, @janowsiany 🙌 |
|
The rule needs to be added to the bottom of the list here: https://github.com/sindresorhus/eslint-plugin-unicorn#rules |
|
Should this rule allow |
|
@sindresorhus @MrHen thanks for review |
|
Thank you, @janowsiany :) |
| "unicorn/prefer-add-event-listener": "error", | ||
| "unicorn/no-console-spaces": "error" | ||
| "unicorn/no-console-spaces": "error", | ||
| "no-unreadable-array-destructuring": "error" |
There was a problem hiding this comment.
Isn't this line missing a "unicorn/" prefix?
5 participants
At the moment the rule disallows more than one ignored value in series. However I can tune this, I can allow to ignore at most one value at the beginning if you find current behaviour too mild.
Fixes #197