Skip to content

Fix the e2e test - #699

Merged
gaearon merged 2 commits into
react:masterfrom
fson:fix-e2e
Sep 22, 2016
Merged

Fix the e2e test#699
gaearon merged 2 commits into
react:masterfrom
fson:fix-e2e

Conversation

@fson

@fsonfson commented Sep 22, 2016

Copy link
Copy Markdown
Contributor

In the end-to-end test, we want to test all the latest packages from
the monorepo instead of the versions from npm.

To do this we will now:

  • run lerna bootstrap in the repo root folder before packaging react-scripts
  • bundle the deps like we do for releases, so the e2e test uses those packages

In the end-to-end test, we want to test all the latest packages from
the monorepo instead of the versions from npm.
To do this we will now:
* run `lerna bootstrap` in the repo root folder before packaging react-scripts
* bundle the deps like we do for releases, so the e2e test uses those packages
@gaearon
gaearon merged commit d3ecd6d into react:masterSep 22, 2016
@gaearon

Copy link
Copy Markdown
Contributor

OK, let’s give it a try. Thanks!

@ghostghost added the CLA Signed label Sep 22, 2016
@fson
fson deleted the fix-e2e branch September 22, 2016 18:10
@fsonfson mentioned this pull request Sep 22, 2016
feiqitian pushed a commit to feiqitian/create-react-app that referenced this pull request Oct 25, 2016
* Fix the e2e test
In the end-to-end test, we want to test all the latest packages from
the monorepo instead of the versions from npm.
To do this we will now:
* run `lerna bootstrap` in the repo root folder before packaging react-scripts
* bundle the deps like we do for releases, so the e2e test uses those packages
* Fix copy-pasted comment
@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.

3 participants

@fson@gaearon@facebook-github-bot