search

sasstools / sass-lint

Pure Node.js Sass linting
MIT License
1.77k stars 534 forks source link

Implement new rule `no-excessive-empty-lines` (fixes #1015) #1294

Open fiso opened 4 years ago

fiso commented 4 years ago

What do the changes you have made achieve? Implements new rule no-excessive-empty-lines. The rule is configurable to allow or disallow consecutive empty lines, and blank lines at the start and end of blocks, respectively. A couple of minor quality-of-life type improvements have also been included in the PR, since I couldn't keep my hands off of them (commits 9653690 and 982ac82).

Are there any new warning messages? No.

Have you written tests? Yes.

Have you included relevant documentation Yes.

Which issues does this resolve?

1015 #1093 #1073

<DCO 1.1 Signed-off-by: Filip Haglund filip.l.haglund@gmail.com>

fiso commented 4 years ago

Is there anything I can do to fix the failing Appveyor build? I can't see any logs so I have no clue why it's failing. Could this be a problem with the CI setup, or are my changes causing it?