-
Notifications
You must be signed in to change notification settings - Fork 455
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
Let babel-jest handle loading babel config #1370
Conversation
Pull Request Test Coverage Report for Build 3951
💛 - Coveralls |
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.
This looks good to me. What problems do you anticipate with this change @ahnpnl ?
we now only collect any babel configs we can and pass them to |
ok. Let's merge this in then. I'll push a new release later today after that |
ok let's proceed like that |
pushing a release out shortly. aside: @ahnpnl when you get the chance, could you please check the message I've sent on slack? |
Yup, letting |
Closes #1341
Note: this needs to be well tested because it looks to me like a breaking change.