New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix empty lines rules #4260
Fix empty lines rules #4260
Conversation
report({ | ||
message: messages.expected(max), | ||
node: decl, | ||
index: placeIndexOnValueStart(decl) + node.sourceIndex, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
so for function-max-empty-lines
column will be set to start of the function
a { transform: translate() }
/* ↑ */
/* column will start here */
report({ | ||
message: messages.expected(max), | ||
node: rule, | ||
index: 0, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for selector-max-empty-lines
column set to start of selector
report({ | ||
message: messages.expected(max), | ||
node: decl, | ||
index: 0, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for value-list-max-empty-lines
column set to start of the property
@vankop could you help summarize this PR in one sentence for commit message and changelog entry, please? :) |
Rework |
In Changelog parlance:
|
|
related to #4232 . Now there is no extra warnings.
fix:
function-max-empty-lines
selector-max-empty-lines
value-list-max-empty-lines
Only for
value-list-max-empty-lines
was test case with comments with exited max empty lines length, so only in the end of this work I realized that comments inside rule range are skipped =(Also there is no info in README about it
If this behavior must be restored I can rework