Add option `minimumLines` to ignore `emptyLineBefore` if group has less than specified amount of properties

[hudochenkov]

Add option `minimumLines` to ignore `emptyLineBefore` if group has less than specified amount of properties

[hudochenkov]

I'm not sure minimumLines option is viable.

What it should do if there less properties than minimumLines? Ignore emptyLineBefore for this group or revert it?

If emptyLineBefore will be ignored, with a config above this won't be considered as warning:

p {
	position: relative;
	display: block;
	font-family: sans-serif;
}

p {
	position: relative;
	display: block;

	font-family: sans-serif;
}

If emptyLineBefore will be reverted, with a config above this won't be considered as warning:

p {
	position: relative;
	display: block;
	font-family: sans-serif;
}

This will be a warning:

p {
	position: relative;
	display: block;

	font-family: sans-serif;
}

With the following config and if emptyLineBefore will be reverted:

{
	"minimumLines": 2,
	"emptyLineBefore": "never",
	"properties": [
		"position",
		"display"
	]
},
{
	"minimumLines": 2,
	"emptyLineBefore": "never",
	"properties": [
		"font",
		"color"
	]
}

The following patterns are considered warnings:

p {
	position: relative;
	display: block;
	font-family: sans-serif;
}

The following patterns are not considered warnings:

p {
	position: relative;
	display: block;

	font-family: sans-serif;
}

[thasmo]

Maybe it should just be ignored, so it's allowed to skip the empty line but it will be no error with the empty line?

[hudochenkov]

Sorry, this feature will never be implemented, because of rare use cases and it will lead to increased code complexity without real benefits.