Skip to content
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

Support plugin/preset definitions without prefixes in .babelrc (#303) #305

Merged
merged 1 commit into from
Nov 1, 2016

Conversation

zdoc01
Copy link
Contributor

@zdoc01 zdoc01 commented Oct 29, 2016

Resolves #303. Still supports babel-* plugins and presets in .babelrc, as well as without the prefixes. Also still supports relative and absolute paths.

@delambo
Copy link
Member

delambo commented Oct 31, 2016

@zdoc01 - Can you quickly look into making a unit test for this. I think we stub out the babelrc so this may be hard to test. Since this is pretty simple and will only temporarily live in the project I'm not terribly concerned about a test but if you find that it is easy to add, please do. Also, feel free to do it in a follow-up pull request.

@delambo delambo merged commit e6a044f into nytimes:master Nov 1, 2016
@delambo
Copy link
Member

delambo commented Nov 1, 2016

Closed since I know you're working against the clock @zdoc01. Please see my comment above and followup with a pull if it's possible.

@zdoc01
Copy link
Contributor Author

zdoc01 commented Nov 1, 2016

@delambo - Sorry, I didn't get a chance to respond yesterday. I'll happily take a look at adding a test when I get the chance. May not be today, but hopefully tomorrow at the latest.

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.

Support plugin/preset definitions without prefixes in .babelrc
3 participants