Skip to content

Conversation

zpao
Copy link
Member

@zpao zpao commented Oct 5, 2016

Fixes both parts of #7880 and then the same for react-native-renderer.

  • ensures we packages lib/ in react-test-renderer
  • make sure we run npm pack on the compiled packages, not the original src.

cc @aweary @spicyj

zpao added 2 commits October 5, 2016 13:58
We were running this on the original packages not the compiled ones, resulting in missing files
@sophiebits
Copy link
Collaborator

lgtm

@zpao zpao merged commit 077d660 into facebook:master Oct 5, 2016
@zpao zpao modified the milestones: 15.4.0, 15-next Oct 5, 2016
zpao added a commit that referenced this pull request Oct 5, 2016
* Ensure lib/ is packaged for react-test-renderer

* Run npm pack from right working directory

We were running this on the original packages not the compiled ones, resulting in missing files

(cherry picked from commit 077d660)
acusti pushed a commit to brandcast/react that referenced this pull request Mar 15, 2017
* Ensure lib/ is packaged for react-test-renderer

* Run npm pack from right working directory

We were running this on the original packages not the compiled ones, resulting in missing files
NMinhNguyen referenced this pull request in enzymejs/react-shallow-renderer Jan 29, 2020
* Ensure lib/ is packaged for react-test-renderer

* Run npm pack from right working directory

We were running this on the original packages not the compiled ones, resulting in missing files
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.

3 participants