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
Docs: On maxEOF with eol-last (fixes #12742) #13374
Conversation
Added an explanation on how and why maxEOF should be set to 0 to work with the eol-last rule.
Added an explanation on how and why maxEOF should be set to 0 to work with the eol-last rule.
|
It was based on the discussion on the issue, so I thought it was worth mentioning. If you prefer, I can remove the mention to the Airbnb style guide (which requires the use of the |
Yeah, since we're talking about two core rules here, I think it's best not to mention the Airbnb style guide. I think it's confusing, since it makes it sound like |
Added an explanation on how and why maxEOF should be set to 0 to work with the eol-last rule.
Done. Let me know if there are any other changes required. |
Added an explanation on how and why maxEOF should be set to 0 to work with the eol-last rule.
Added an explanation on how and why maxEOF should be set to 0 to work with the eol-last rule.
Added an explanation on how and why maxEOF should be set to 0 to work with the eol-last rule.
Prerequisites checklist
What is the purpose of this pull request? (put an "X" next to an item)
[X] Documentation update
[ ] Bug fix (template)
[ ] New rule (template)
[ ] Changes an existing rule (template)
[ ] Add autofixing to a rule
[ ] Add a CLI option
[ ] Add something to the core
[ ] Other, please explain:
What changes did you make? (Give an overview)
I added a section in the documentation for the rule
no-multiple-empty-lines
explaining how and whymaxEOF
to zero when using the Airbnb style guide ruleeol-last
.Is there anything you'd like reviewers to focus on?