Add a rule to prevent the use of labels

[DavidBruant]

They're usually a bad practice.

Maybe related, I'm not sure I fully understand no-empty-label. I need to read more about it.

[nzakas]

Happy to consider it. Do you have more details as to why this is a bad practice? (We like to include narrative in the docs for each rule)

[DavidBruant]

Pretty much all concerns about GOTO apply to labels.
We got rid of GOTO by adding higher-level primitives like while loops, if/else, switch statements and function calls which cover the vast majority of GOTO use cases. These constructs make code easier to read/review/refactor than code using labels.

I don't hold any sort of proof, but given the sort of flexibility they provide, it's possible the use of labels locally disables some optimizations of current JS engines. At the very least, the fact that they're rarely used and absent from popular benchmarks makes them unlikely target for JS engine engineers to optimize for.

[nzakas]

Working on this.