LineLength is calculated from logical lines instead of from screen lines

[oliverklee]

This problem might related to issue #25. It occurs with haml-lint 0.6.0 and rubocop 0.25.0.

For testing the issue, I have set the maximum line length to 200 both in my rubocop configuration as well as my haml-lint configuration. I get this error for the following HAML lines:

klee@gonzales:~/eclipse/justworkshops$ haml-lint app/views/
app/views/home/admin.html.haml:118 [W] Line is too long. [358/200]

This is the corresponding piece of HAML code:

  %p!= t 'views.home.admin.statisticsText1Html',
    users: number_with_delimiter(5_392, delimiter: t('views.thousandsSeperator')),
    attendees: number_with_delimiter(4_966, delimiter: t('views.thousandsSeperator')),
    offerers: number_with_delimiter(554, delimiter: t('views.thousandsSeperator')),
    admins: number_with_delimiter(4, delimiter: t('views.thousandsSeperator'))

It seems that HAML (or RuboCop?) check the length of the logical line (consisting of 5 screen lines) instead of the lengths of the individual screen lines.

[sds]

Yes, this is a bug due to the namespace change of Rubocop's LineLength cop. I've fixed this in c155377 and released a new version 0.6.1.

Thanks for the report!

[oliverklee]

❤️