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

add dependencies to react-test-renderer and react-addons #8467

Merged
merged 1 commit into from
Dec 1, 2016

Conversation

kweiberth
Copy link
Contributor

What and Why:

  • When using npm version 2, object-assign and fbjs were not installed
  • This PR adds object-assign and fbjs as explicit dependencies to both react-test-renderer and react-addons

**What** and **Why**:

* When using npm version 2, `object-assign` and `fbjs` were not getting properly installed
* This PR adds `object-assign` and `fbjs` as explicit dependencies to both `react-test-renderer` and `react-addons`
@kweiberth
Copy link
Contributor Author

Does react-addons for sure need fbjs as a dependency?

@gaearon
Copy link
Collaborator

gaearon commented Nov 30, 2016

Can you show me what is test renderer using from fbjs?

@kweiberth
Copy link
Contributor Author

Cannot find module 'fbjs/lib/invariant' from 'ReactComponentEnvironment.js'

      at Resolver.resolveModule (node_modules/jest/node_modules/jest-cli/node_modules/jest-resolve/build/index.js:151:17)
      at Object.<anonymous> (node_modules/react-test-renderer/lib/ReactComponentEnvironment.js:16:17)
      at Object.<anonymous> (node_modules/react-test-renderer/lib/ReactTestRenderer.js:21:33)

@gaearon
Copy link
Collaborator

gaearon commented Nov 30, 2016

Oh right, it ships with a complete renderer copy now..
I think there is no harm in adding fbjs to addons too just in case.
If anything it would be consistent.

@gaearon gaearon self-assigned this Nov 30, 2016
@gaearon gaearon added this to the 15-next milestone Dec 1, 2016
@gaearon gaearon merged commit 7cd2602 into facebook:master Dec 1, 2016
@gaearon
Copy link
Collaborator

gaearon commented Dec 1, 2016

LGTM

@gghez
Copy link

gghez commented Dec 7, 2016

react-test-renderer v15.4.1 is the latest and does not include this fix. How is it planned to be released?

@gaearon
Copy link
Collaborator

gaearon commented Jan 6, 2017

It will be released as part of 15.4.2 (shortly).

@gaearon gaearon modified the milestones: 15-hipri, 15-lopri, 15.4.2 Jan 6, 2017
gaearon pushed a commit that referenced this pull request Jan 6, 2017
**What** and **Why**:

* When using npm version 2, `object-assign` and `fbjs` were not getting properly installed
* This PR adds `object-assign` and `fbjs` as explicit dependencies to both `react-test-renderer` and `react-addons`
(cherry picked from commit 7cd2602)
@gaearon
Copy link
Collaborator

gaearon commented Jan 6, 2017

Should be fixed if you update all packages to 15.4.2.
Thank you!

richardgiddings pushed a commit to richardgiddings/menu that referenced this pull request Jan 9, 2017
richardgiddings pushed a commit to richardgiddings/menu that referenced this pull request Jan 9, 2017
acusti pushed a commit to brandcast/react that referenced this pull request Mar 15, 2017
**What** and **Why**:

* When using npm version 2, `object-assign` and `fbjs` were not getting properly installed
* This PR adds `object-assign` and `fbjs` as explicit dependencies to both `react-test-renderer` and `react-addons`
laurinenas pushed a commit to laurinenas/react that referenced this pull request May 28, 2018
**What** and **Why**:

* When using npm version 2, `object-assign` and `fbjs` were not getting properly installed
* This PR adds `object-assign` and `fbjs` as explicit dependencies to both `react-test-renderer` and `react-addons`
NMinhNguyen referenced this pull request in enzymejs/react-shallow-renderer Jan 29, 2020
**What** and **Why**:

* When using npm version 2, `object-assign` and `fbjs` were not getting properly installed
* This PR adds `object-assign` and `fbjs` as explicit dependencies to both `react-test-renderer` and `react-addons`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants