Skip to content

[init] Honor the version of "react" under peerDeps when setting up a new project - #6846

Closed
ide wants to merge 1 commit into
react:masterfrom
expo:react-peer-dep
Closed

ide wants to merge 1 commit into
react:masterfrom
expo:react-peer-dep

Conversation

@ide

@ide ide commented Apr 6, 2016

Copy link
Copy Markdown
Contributor

We need this since React 15.0.0 is coming and will break react-native init, which currently installs the latest version of React. We'll need some changes to React Native to support 15 that Sebastian is actively working on, but till that lands we want react-native init to continue working.

Test Plan: Run scripts/test-manual-e2e.sh and verify that it ran npm install react@^0.14.5 --save instead of npm install react --save.

…project

We need this since React 15.0.0 is coming and will break `react-native init`, which currently installs the latest version of React. We'll need some changes to React Native to support 15 that Sebastian is actively working on, but till that lands we want `react-native init` to continue working.

Test Plan: Run `scripts/test-manual-e2e.sh` and verify that it ran `npm install react@^0.14.5 --save` instead of `npm install react --save`.
@ide ide changed the title [init] Honor the version of "react" under peerDeps when setting up a … [init] Honor the version of "react" under peerDeps when setting up a new project Apr 6, 2016
@ide

ide commented Apr 6, 2016

Copy link
Copy Markdown
Contributor Author

Will cherry-pick this into 0.24 and we might want this in 0.23 too so that the release React 15 doesn't break things... cc @mkonicek

@facebook-github-bot

Copy link
Copy Markdown
Contributor

By analyzing the blame information on this pull request, we identified @skevy, @mkonicek and @foghina 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 Apr 6, 2016
@ide ide mentioned this pull request Apr 6, 2016
4 tasks done
@zpao zpao mentioned this pull request Apr 6, 2016
13 of 15 tasks
@sebmarkbage

Copy link
Copy Markdown
Contributor

I think this looks good and since it only affects open source builds it shouldn't break any of our stuff. So I'll just land it for now.

@facebook-github-bot shipit

@facebook-github-bot

Copy link
Copy Markdown
Contributor

Thanks for importing. If you are an FB employee go to Phabricator to review.

@ide

ide commented Apr 7, 2016

Copy link
Copy Markdown
Contributor Author

@facebook-github-bot shipit

@facebook-github-bot

Copy link
Copy Markdown
Contributor

Thanks for importing. If you are an FB employee go to Phabricator to review.

1 similar comment
@facebook-github-bot

Copy link
Copy Markdown
Contributor

Thanks for importing. If you are an FB employee go to Phabricator to review.

@ghost ghost closed this in c30d125 Apr 7, 2016
ide added a commit that referenced this pull request Apr 7, 2016
…ject

Summary:We need this since React 15.0.0 is coming and will break `react-native init`, which currently installs the latest version of React. We'll need some changes to React Native to support 15 that Sebastian is actively working on, but till that lands we want `react-native init` to continue working.
Closes #6846

Differential Revision: D3148182

Pulled By: sebmarkbage

fb-gh-sync-id: 3df5bc184c0b82d2c9c320c620256c7c8568d90b
fbshipit-source-id: 3df5bc184c0b82d2c9c320c620256c7c8568d90b
@zpao

zpao commented Apr 7, 2016

Copy link
Copy Markdown
Contributor

What's the status on shipping this?

@ide

ide commented Apr 7, 2016

Copy link
Copy Markdown
Contributor Author

We've shipped this fix in 0.24.0-rc3 and still need to ship it in 0.23.1. Do you want to discuss the release schedule on IRC?

@ide
ide deleted the react-peer-dep branch April 7, 2016 19:14
ide added a commit that referenced this pull request Apr 7, 2016
…ject

Summary:We need this since React 15.0.0 is coming and will break `react-native init`, which currently installs the latest version of React. We'll need some changes to React Native to support 15 that Sebastian is actively working on, but till that lands we want `react-native init` to continue working.
Closes #6846

Differential Revision: D3148182

Pulled By: sebmarkbage

fb-gh-sync-id: 3df5bc184c0b82d2c9c320c620256c7c8568d90b
fbshipit-source-id: 3df5bc184c0b82d2c9c320c620256c7c8568d90b
@ide

ide commented Apr 7, 2016

Copy link
Copy Markdown
Contributor Author

RN 0.23.1 is ready for 15.0: https://github.com/facebook/react-native/releases/tag/v0.23.1

zebulgar pushed a commit to nightingale/react-native that referenced this pull request Jun 18, 2016
…ject

Summary:We need this since React 15.0.0 is coming and will break `react-native init`, which currently installs the latest version of React. We'll need some changes to React Native to support 15 that Sebastian is actively working on, but till that lands we want `react-native init` to continue working.
Closes react#6846

Differential Revision: D3148182

Pulled By: sebmarkbage

fb-gh-sync-id: 3df5bc184c0b82d2c9c320c620256c7c8568d90b
fbshipit-source-id: 3df5bc184c0b82d2c9c320c620256c7c8568d90b
grabbou pushed a commit to react-native-community/cli that referenced this pull request Sep 26, 2018
…ject

Summary:We need this since React 15.0.0 is coming and will break `react-native init`, which currently installs the latest version of React. We'll need some changes to React Native to support 15 that Sebastian is actively working on, but till that lands we want `react-native init` to continue working.
Closes react/react-native#6846

Differential Revision: D3148182

Pulled By: sebmarkbage

fb-gh-sync-id: 3df5bc184c0b82d2c9c320c620256c7c8568d90b
fbshipit-source-id: 3df5bc184c0b82d2c9c320c620256c7c8568d90b
This pull request was closed.
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