apache / cordova-plugman

Apache Cordova Plugman
https://cordova.apache.org/
Apache License 2.0
399 stars 151 forks source link

[CB-4714] Add -f option to 'plugin rm' to forcefully remove a plugin. #22

Closed jbondc closed 10 years ago

jbondc commented 10 years ago

I got rid of fs.statSync(pluginDir).isDirectory() since that can fail if the directory doesn't exist.

agrieve commented 10 years ago

Thanks for the pull requests. Have you looked at adding a unit test for this? spec/uninstall.spec.js would be the place for it.

jbondc commented 10 years ago

Will try to get setup with spec tests this Friday