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

[Babel] Support plugins that conform to ES6 modules #4513

Closed
wants to merge 1 commit into from

Conversation

ide
Copy link
Contributor

@ide ide commented Dec 2, 2015

ES6 modules export an object with a property called default. See Babel itself for how this is handled: babel/babel@b5b7e34

Test Plan: Used a Babel 6 plugin that is implemented as an ES6 module.

ES6 modules export an object with a property called `default`. See Babel itself for how this is handled: babel/babel@b5b7e34

Test Plan: Used a Babel 6 plugin that is implemented as an ES6 module.
@ide ide added the Packager label Dec 2, 2015
@ide
Copy link
Contributor Author

ide commented Dec 2, 2015

cc @martinbigio @tadeuzagallo for review

@facebook-github-bot
Copy link
Contributor

By analyzing the blame information on this pull request, we identified @davidaurelio, @sebmck and @sahrens to be potential reviewers.

@facebook-github-bot facebook-github-bot added GH Review: review-needed CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. labels Dec 2, 2015
@tadeuzagallo
Copy link
Contributor

@facebook-github-bot shipit

@facebook-github-bot
Copy link
Contributor

Thanks for importing. If you are an FB employee go to https://our.intern.facebook.com/intern/opensource/github/pull_request/1665686983713924/int_phab to review.

@mkonicek
Copy link
Contributor

mkonicek commented Dec 4, 2015

This got stuck in Sandcastle (we need reporting!). Landing now :)

@ide ide closed this in dff8f53 Dec 4, 2015
ide added a commit that referenced this pull request Dec 4, 2015
Summary:
ES6 modules export an object with a property called `default`. See Babel itself for how this is handled: babel/babel@b5b7e34
Closes #4513

Reviewed By: svcscm

Differential Revision: D2715512

Pulled By: mkonicek

fb-gh-sync-id: 40e5ea35adcdb66806a4895578d637cd72538619
@ide ide deleted the babel-resolution branch January 3, 2016 01:02
cpojer pushed a commit to facebook/metro that referenced this pull request Jan 26, 2017
Summary:
ES6 modules export an object with a property called `default`. See Babel itself for how this is handled: babel/babel@b5b7e34
Closes facebook/react-native#4513

Reviewed By: svcscm

Differential Revision: D2715512

Pulled By: mkonicek

fb-gh-sync-id: 40e5ea35adcdb66806a4895578d637cd72538619
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants