Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

Copy node_modules if the directory exists #317

Merged
merged 1 commit into from
Dec 19, 2018
Merged

Copy node_modules if the directory exists #317

merged 1 commit into from
Dec 19, 2018

Conversation

erisu
Copy link
Member

@erisu erisu commented Dec 19, 2018

Platforms affected

windows

What does this PR do?

  • Copy the platform node_modules only when the folder exists.

apache/cordova#32

What testing has been done on this change?

  • npm run eslint
  • npm run test-unit

@erisu erisu requested a review from dpogue December 19, 2018 01:07
dpogue
dpogue approved these changes Dec 19, 2018
View reviewed changes
Copy link
Member

@dpogue dpogue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@erisu erisu requested a review from dpogue December 19, 2018 01:34
@erisu erisu merged commit e360d30 into apache:master Dec 19, 2018
@aman2604 aman2604 mentioned this pull request Mar 11, 2019
Closed
@erisu erisu deleted the copy_node_modules branch April 4, 2019 06:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@dpogue dpogue Awaiting requested review from dpogue

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@erisu @dpogue