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
Update: add fixer for lines-around-directive
#7217
Update: add fixer for lines-around-directive
#7217
Conversation
@not-an-aardvark, thanks for your PR! By analyzing the annotation information on this pull request, we identified @kaicataldo to be a potential reviewer |
LGTM |
I'll champion this. @eslint/eslint-team could use two more supporters! |
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.
LGTM. Need to wait for this to be approved and to have some others review.
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.
LGTM
LGTM. Thanks for contributing to ESLint! |
What is the purpose of this pull request? (put an "X" next to item)
[ ] Documentation update
[ ] Bug fix (template)
[ ] New rule (template)
[ ] Changes an existing rule (template)
[x] Add autofixing to a rule
[ ] Add a CLI option
[ ] Add something to the core
[ ] Other, please explain:
Please check each item to ensure your pull request is ready:
What changes did you make? (Give an overview)
This adds a fixer for
lines-around-directive
.If the rule reports a missing newline, the fixer adds the newline. If the rule reports an unexpected newline, the fixer removes the newline.
Is there anything you'd like reviewers to focus on?
Nothing in particular.