Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix the e2e test #699

Merged
merged 2 commits into from
Sep 22, 2016
Merged

Fix the e2e test #699

merged 2 commits into from
Sep 22, 2016

Conversation

fson
Copy link
Contributor

@fson fson commented Sep 22, 2016

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 facebook:master Sep 22, 2016
@gaearon
Copy link
Contributor

gaearon commented Sep 22, 2016

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

@ghost ghost added the CLA Signed label Sep 22, 2016
@fson fson deleted the fix-e2e branch September 22, 2016 18:10
@fson fson 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
@lock lock bot locked and limited conversation to collaborators Jan 22, 2019
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants