Add new type of warning, probably within 6xx range, about lines that are longer than some limit. Limit should be customizable using an option. Some things to consider:
Name of the option: probably max_line_length (--max-line-length in CLI, max line length inline).
Default length limit: 80 is probably too strict, 120 seems OK. Disabling this warning by default is also an option.
Implementation: whitespace.lua should save line lengths to a subtable of check result. cache.lua should ensure the subtable is saved properly. filter.lua should temporarily issue a warning for each line, with line length as a field of warning object. options.lua should filter these warnings based on the value of max_line_length option.
mpeterv
commented
7 years ago
Add new type of warning, probably within
6xxrange, about lines that are longer than some limit. Limit should be customizable using an option. Some things to consider:max_line_length(--max-line-lengthin CLI,max line lengthinline).whitespace.luashould save line lengths to a subtable of check result.cache.luashould ensure the subtable is saved properly.filter.luashould temporarily issue a warning for each line, with line length as a field of warning object.options.luashould filter these warnings based on the value ofmax_line_lengthoption.