Uh oh!
There was an error while loading. Please reload this page.
[npm] Remove unused npm packages and move CI dependency to devDependencies - #13517
[npm] Remove unused npm packages and move CI dependency to devDependencies#13517ide wants to merge 1 commit into
Conversation
hramos
left a comment
There was a problem hiding this comment.
This will need a manual import, and we'll also need to make sure internal tests run. I'm going to import it (without landing).
facebook-github-bot
commented
Apr 27, 2017
@hramos has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
skevy
commented
Jun 8, 2017
@ide this should be either cleaned up or closed, given the recent changes to dependencies with the packager moving. |
ide
commented
Jun 8, 2017
Updated this PR. @hramos could you please re-import this when you have a chance to? |
facebook-github-bot
commented
Jun 9, 2017
@hramos has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
ed32ec1 to
caf085fComparecpojer
commented
Jul 7, 2017
This is awesome! |
hramos
commented
Jul 7, 2017
I'll take a look at this after #14880 lands. |
…ncies There are several npm packages that are no longer used and do not show up when grepping the code base. Also the "async" dependency is used only in CI and should be moved to devDependencies. Test Plan: Load the UIExplorer. Run Travis CI, Circle CI, and FB internal tests.
pull-bot
commented
Aug 1, 2017
This PR has been submitted by a core contributor. Attention: @facebook/react-native Generated by 🚫 dangerJS |
facebook-github-bot
commented
Aug 2, 2017
@hramos has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
There are several npm packages that are no longer used and do not show up when grepping the code base. Also the "async" dependency is used only in CI and should be moved to devDependencies.
Test Plan: Load the UIExplorer. Run Travis CI, Circle CI, and FB internal tests.