-
-
Notifications
You must be signed in to change notification settings - Fork 44
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
Make most variants of module.exports = { ... } fixable #17
Make most variants of module.exports = { ... } fixable #17
Conversation
Both work and satisfy the linter
Seems like in-between comments are only attached to one of the nodes
CI failure ( |
no worries, I've re-run the ci - seems working now! 🎉 |
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.
just a few minor suggestions, thx for the contributing! ~
c0a41f7
to
8425c80
Compare
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, thanks!
released in v15.2.0 |
This is mysticatea#208 against your fork now that
eslint-plugin-node
is unmaintained andeslint-plugin-n
is being used byeslint-config-standard
😇It got reviewed by the original author, who then disappeared before merging it.