search

apache / cordova-plugman

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

Fix orgy #113

Closed raphinesse closed 5 years ago

raphinesse commented 5 years ago

Motivation and Context

Plugman is horribly broken in so many aspects. This PR fixes some of the bugs. All changes are only patch level IMO.

Fixes #111

Description

See commits for details.

Testing

I did manual testing of all but the install and uninstall commands. The one existing test case passes.

raphinesse commented 5 years ago

@dpogue Oh, nevermind the CI. There are no tests anyway 😋

Okay, I'm being unfair here. There is one test case. And it tested the parsing of options that should not even exist before I fixed it here 😁