Skip to content
This repository has been archived by the owner on May 22, 2024. It is now read-only.

Included rimraf package. #26

Merged
merged 4 commits into from
May 3, 2017
Merged

Included rimraf package. #26

merged 4 commits into from
May 3, 2017

Conversation

emilyrohrbough
Copy link
Contributor

Summary

Included rimraf package to be windows compatible. Addresses #23 and cerner/terra-core#281

@cerner/terra

package.json Outdated
@@ -35,7 +35,8 @@
},
"scripts": {
"bootstrap": "lerna bootstrap",
"clean:all": "rm -rf ./node_modules && rm -rf ./packages/*/node_modules",
"test-rim": "rimraf ./test-rim",
Copy link
Contributor

Choose a reason for hiding this comment

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

Whats this line for?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Opps must not have cleaned this up!

I was hitting errors before I changed the order deleting the node_module folders & I don't have rimraf installed globally so I was trying to work it out.

Copy link
Contributor

@bjankord bjankord left a comment

Choose a reason for hiding this comment

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

lgtm

@dkasper-was-taken dkasper-was-taken merged commit bbd3c8c into master May 3, 2017
@dkasper-was-taken dkasper-was-taken deleted the use-rimraf branch May 3, 2017 21:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants