Reviewed, left comments. Overall looks good, thanks Mark.
However, seems you left something uncommitted, since you literally cannot
require('cordova-lib'); without an exception thrown. Yet all the
cordova-lib tests pass. What does it say about the quality of the tests ;)
On Tue, Mar 10, 2015 at 6:24 PM, Jesse <[email protected]> wrote:
> Had a quick look, looks great!
>
> @purplecabbage
> risingj.com
>
> On Tue, Mar 10, 2015 at 3:09 PM, Mark Koudritsky <[email protected]>
> wrote:
>
> > I've created a pull request with changes across cordova-lib that should
> > make it much easier to consolidate the platform specific code in
> > cordova-lib to reduce duplication and maybe eventually move this code to
> > the platforms repos.
> >
> > https://github.com/apache/cordova-lib/pull/183
> > Feedback is welcome.
> >
> > Those changes only deal with interfacing the platform specific code from
> > cordova/metadata/ and plugman/platforms/ dirs, the dirs themselves and
> > their contents weren't changed yet.
> >
>
