Skip to content

Add support for eslint-plugin-babel#72

Merged
lydell merged 2 commits intoprettier:masterfrom
silvenon:eslint-plugin-babel
Jan 19, 2019
Merged

Add support for eslint-plugin-babel#72
lydell merged 2 commits intoprettier:masterfrom
silvenon:eslint-plugin-babel

Conversation

@silvenon
Copy link
Copy Markdown
Contributor

I'm not aware of any popular ESLint configs that use it so I just left it out of the configuration example for the sake of simplicity, but I think it's visible enough that people will notice that it exists.

Fixes #67.

@silvenon
Copy link
Copy Markdown
Contributor Author

Will fix tests tomorrow.

@silvenon
Copy link
Copy Markdown
Contributor Author

Ok, done. I made an exception in the tests when checking for "babel" because I didn't include eslint-plugin-babel it in the configuration example.

Btw, the test suite is awesome 👌

Copy link
Copy Markdown
Member

@lydell lydell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good!

I left a few comments. If some of them are too nitpicky – let me know and I'll fix them myself.

Comment thread .eslintrc.base.js Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread package.json Outdated
Comment thread test/rules.test.js Outdated
@silvenon
Copy link
Copy Markdown
Contributor Author

silvenon commented Jan 18, 2019

Ok, let me know if I missed something. Feel free to be nitpicky, I like neatness as well 😉

@lydell lydell merged commit 2297f5a into prettier:master Jan 19, 2019
@lydell
Copy link
Copy Markdown
Member

lydell commented Jan 19, 2019

Thanks! 🎉

@silvenon silvenon deleted the eslint-plugin-babel branch January 20, 2019 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants