New Rule Proposal: Disallow multiple empty lines

[0xGuybrush]

Hello,

I'd like to propose a rule to disallow multiple consecutive empty lines in a file. E.g.:

/* Allowed */
.foo {
    margin: 5px;
}

.bar {
    padding: 10px;
}

/* Disallowed */
.foo {
    margin: 5px;
}



.bar {
    padding: 10px;
}

For JavaScript, we're using ESLint's no-multiple-empty-lines rule which allows for:

• Max number of allowed consecutive empty lines.
• Separate max configurable for start & end of files.

I'd be happy to try and submit a PR / test cases for same, but wanted to first check if it sounded like a feature others would want to.

Please let me know if I can give more relevant details.

Thanks!
Dave

[DanPurdy]

Hi sorry for the delay in getting back to you, had no free time it seems recently. I'd be happy to see a PR submitted for this. Thanks!

[fiso]

I would love this too.

[0xGuybrush]

Cool, great! I'll try to look into this this week & see if I can get something raised for it.

[alfaproject]

I would love this as well. (:

[fiso]

For anyone else who feel like they need this rule, I now have a fork where this rule is implemented: https://github.com/fiso/sass-lint

You can try it out by changing your dependency import line to say "sass-lint": "fiso/sass-lint#develop" and then running npm install.

I'd be happy to submit a PR for it but I feel like I'd want some eyes on my take before I do that, seeing as how I'm completely new to this codebase and all.

[fiso]

FWIW my team has been using the forked version for a couple of weeks now and encountered no problems with it.

[Potherca]

There seems to be some overlap between this suggested rule and the one suggested in #1073.

[fiso]

FWIW the rule I implemented in my fork (linked above) supports the suggestion from #1073 as well (optionally).

[Potherca]

@fiso Awesome! So.. Do you think we should consider #1093 and #1073 as duplicates of this issue?

Also... it might be high time to open that PR! 👍

[fiso]

@Potherca Yes that would make sense.

As for the pull request, yes, I've been meaning to do that since forever. I keep putting it off since the checklist for submitting PRs to the project seems kind of daunting. But I'll make time for it.