Our eslint config doesn't enforce line length in any way, resulting in inconsistent code, and comments, which hurts readability.
Manual:Coding_conventions/JavaScript#Line_length mentions 80-100 as a convention.
The JavaScript community at large has standardized at 80 chars length, some examples:
- eslint's rule defaults to 80
Although there are some famous presets that use 100:
In addition, we should special case some things like //eslint-disable* comments, and QUnit lines which tend to run over because they contain a long string. Other things like lines with a string, template literals, long urls, ... could also be taken into account, see example