Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

allowDestructuring option for no-sparse-arrays rule

See original GitHub issue

What rule do you want to change? no-sparse-arrays

Does this change cause the rule to produce more or fewer warnings? fewer warnings

How will the change be implemented? (New option, new default behavior, etc.)? New option

Please provide some example code that this change will affect:

const [, y] = [1, 2];

What does the rule currently do for this code? error Unexpected comma in middle of array no-sparse-arrays

What will the rule do after it’s changed? it’ll allow it, with "no-sparse-array": ["error", {"allowDestructuring": true}]

Are you willing to submit a pull request to implement this change? I can try, yes

Issue Analytics

State:
Created 4 years ago
Comments:5 (3 by maintainers)

Top GitHub Comments

1reaction

mdjermanoviccommented, Oct 19, 2019

Hi @caub, thanks for the issue!

This would be a bug, but I can’t reproduce it: Online Demo Link

It might be good to fill the bug report issue template.

1reaction

ljharbcommented, Oct 19, 2019

This shouldn’t require an option - it’s broken, since omitting items in array destructuring is in no way “sparse arrays”. This rule simply shouldn’t consider array destructurings.