raphinesse opened a new pull request #162: URL: https://github.com/apache/cordova-common/pull/162
### Motivation and Context <!-- Why is this change required? What problem does it solve? --> <!-- If it fixes an open issue, please link to the issue here. --> Config munging is a core aspect of Cordova and a source of many issues for us. When reading the code to find causes of reported issues (like #156), I noticed that much of the existing code is really verbose and repetitive, making it hard to grasp what's going on. So I went ahead and refactored the whole thing a bit, hopefully making it more approachable in the future. ### Description <!-- Describe your changes in detail --> Hopefully, the commit messages provide enough context to see what's going on. I took great care to make the diffs most obvious whenever possible. ### Testing <!-- Please describe in detail how you tested your changes. --> The existing unit tests still pass with one small change pertaining to an implementation detail. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
