Skip to content

Extract Babel configuration to babel-preset-react-app - #701

Merged
gaearon merged 1 commit into
react:masterfrom
fson:babel-preset
Sep 22, 2016
Merged

Extract Babel configuration to babel-preset-react-app#701
gaearon merged 1 commit into
react:masterfrom
fson:babel-preset

Conversation

@fson

@fsonfson commented Sep 22, 2016

Copy link
Copy Markdown
Contributor

Discussed in #674.

Test plan:
Verified that npm start, npm run build and npm test work after reinstalling the packages.

@ghostghost added the CLA Signed label Sep 22, 2016
@gaearongaearon added this to the 0.5.0 milestone Sep 22, 2016
@gaearon

Copy link
Copy Markdown
Contributor

I think it is failing because of #699.

The reason I wasn’t using bundledDependencies in e2e before is precisely that: Travis runs out of memory trying to bundle them.

@ghostghost added the CLA Signed label Sep 22, 2016
@fson

fson commented Sep 22, 2016

Copy link
Copy Markdown
ContributorAuthor

How should we fix that? I added bundledDependencies to e2e so it would use the packages from the repo – otherwise npm pack discards the node_modules and create-react-app will install our packages from npm.

@gaearon

Copy link
Copy Markdown
Contributor

Can we make it only bundle our packages?

@gaearon
gaearon merged commit 5b85a36 into react:masterSep 22, 2016
@gaearon

Copy link
Copy Markdown
Contributor

Going to get this in and figure out e2e later.

@fson
fson deleted the babel-preset branch September 22, 2016 19:18
@fsonfson mentioned this pull request Sep 22, 2016
@gaearongaearon mentioned this pull request Sep 22, 2016
feiqitian pushed a commit to feiqitian/create-react-app that referenced this pull request Oct 25, 2016
@locklockBot locked and limited conversation to collaborators Jan 22, 2019
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@fson@gaearon