31
97%
main: 97%

Ran 23 Jun 2026 03:39AM UTC

Files 202

Run time 7s

Badge

Embed ▾

Committed 23 Jun 2026 03:37AM UTC coverage: 96.977% (-0.03%) from 97.007%

Job # 28000422921.31

Build Type

push

github

Committed by

web-flow

Commit Message

Fix LineLength not being disabled inside filter blocks (#658)

A `haml-lint:disable LineLength` comment had no effect on long lines
inside filter blocks such as `:javascript` or `:css`.

`Linter::LineLength` resolves each over-long line to its owning node via
`RootNode#node_for_line` and checks the directive chain on that node. A
`FilterNode` sits on the `:filtername` declaration line while its content
lives on the following lines, but the base `Node#line_numbers` assumed the
text started on the node's own line. The filter's content lines were
therefore left uncovered, so `node_for_line` fell through to a `NullNode`
(which carries no directives) and the disable comment was silently lost.

Override `line_numbers` in `FilterNode` so its span covers the declaration
line plus all of its indented content lines.

Fixes #623

Co-authored-by: Shane da Silva <shane@dasilva.io>

Coverage Stats

6032 of 6220 relevant lines covered (96.98%)

321.19 hits per line

sds / haml-lint / 28000422921 / 31
97%
main: 97%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job ruby4.0-haml6.1-ubuntu - 28000422921.31

sds / haml-lint / 28000422921 / 31 97% main: 97%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job ruby4.0-haml6.1-ubuntu - 28000422921.31

sds / haml-lint / 28000422921 / 31
97%
main: 97%

README BADGES
x