Jesse wanted to look into it a bit more, so probably this will be merged at the end of this week.
Best regards, Vladimir. -----Original Message----- From: Anis KADRI [mailto:[email protected]] Sent: Wednesday, August 26, 2015 11:27 AM To: [email protected] Subject: Re: Cordova-lib refactoring implementation Do it! On Tue, Aug 25, 2015 at 5:09 PM Vladimir Kotikov (Akvelon) < [email protected]> wrote: > Hey, guys. The PRs has been reviewed by Mark and Sergey. > If nobody has any objections, I would like to merge this in tomorrow. > ------------------------ > Best regards, Vladimir. > > -----Original Message----- > From: Vladimir Kotikov (Akvelon) [mailto:[email protected]] > Sent: Friday, August 21, 2015 6:57 PM > To: [email protected] > Subject: RE: Cordova-lib refactoring implementation > > Oops, previous email sent by mistake ☹ > > Hi, guys. > > I’ve submitted a PR, that refactors a cordova-lib API and implements a > PlatformApi polyfill (as discussed in > https://na01.safelinks.protection.outlook.com/?url=https%3a%2f%2fgithu > b.com%2fcordova%2fcordova-discuss%2fpull%2f12&data=01%7c01%7cv-vlkoti% > 40064d.mgd.microsoft.com%7c9ff61bc6ceae43538c1008d2aa420cba%7c72f988bf > 86f141af91ab2d7cd011db47%7c1&sdata=Fd24ZInammyzOtVRlb2qBvHNxaohF9ms50j > 3dhIAkGs%3d > ) > Please find it here: > https://na01.safelinks.protection.outlook.com/?url=https%3a%2f%2fgithu > b.com%2fapache%2fcordova-lib%2fpull%2f282.&data=01%7c01%7cv-vlkoti%400 > 64d.mgd.microsoft.com%7c9ff61bc6ceae43538c1008d2aa420cba%7c72f988bf86f > 141af91ab2d7cd011db47%7c1&sdata=PzMbKu4Hb9Oe4NfkwBsIOZcwfcl31Oq%2fU%2f > VdqBwlqgk%3d To test this you'll also need to check out corresponding > PR to > cordova-cli: > https://na01.safelinks.protection.outlook.com/?url=https%3a%2f%2fgithu > b.com%2fapache%2fcordova-cli%2fpull%2f220.&data=01%7c01%7cv-vlkoti%400 > 64d.mgd.microsoft.com%7c9ff61bc6ceae43538c1008d2aa420cba%7c72f988bf86f > 141af91ab2d7cd011db47%7c1&sdata=IUZpWxvf1bu1x7Jb6UYLrdW10G6bMmvJ8Kl09% > 2fq0Mtk%3d > > This is a first step of refactoring, which includes cordova-lib > refactoring only. > The next steps once this PR will be reviewed and merged: > * Move shared code for platforms/cordova-lib into cordova-common > module > * Prepare and publish cordova-common (this also includes coho update) > * Switch cordova-lib to use cordova-common > > Brief overview of this PR: > It implements PlatformApiPoly class according to PlatformApi spec, > which allows to: > * create/update platform > * execute platform's actions (build/run/add/update) > * do a prepare (needed for CLI workflow only) > * install/uninstall plugins > > Other noticeable changes: > * removes `getPlatformProject` and PlatformProject method/class in > favor of PlatformApiPoly/getPlatformApi > * make assets and js-modules installing/uninstalling through ActionStack > * refactor configChanges to not require plugins_dir in constructor > * moves mergeXml helper to xml-helpers > > I would appreciate any feedback on this PR. > ------------------------ > Best regards, Vladimir. > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] >
