Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cordova.apache.org
MIME-Version: 1.0
References: <356ced51945e4dadb40fbf851406160f@SINPR30MB0057.064d.mgd.msft.net>
 <72bf9f7206ce4953a4e3250acf1cd36c@SINPR30MB0057.064d.mgd.msft.net>
In-Reply-To: 
 <72bf9f7206ce4953a4e3250acf1cd36c@SINPR30MB0057.064d.mgd.msft.net>
From: Anis KADRI <anis.kadri@gmail.com>
Date: Wed, 26 Aug 2015 08:26:45 +0000
Message-ID: 
 <CANpVHHyUA-EjYPvjVragzavUiF3H9K9D6dC=uq5jex8L7UP_5Q@mail.gmail.com>
Subject: Re: Cordova-lib refactoring implementation
To: "dev@cordova.apache.org" <dev@cordova.apache.org>
Content-Type: multipart/alternative; boundary=047d7b67054d0566ba051e329d54

--047d7b67054d0566ba051e329d54
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Do it!

On Tue, Aug 25, 2015 at 5:09 PM Vladimir Kotikov (Akvelon) <
v-vlkoti@microsoft.com> 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:v-vlkoti@microsoft.com]
> Sent: Friday, August 21, 2015 6:57 PM
> To: dev@cordova.apache.org
> Subject: RE: Cordova-lib refactoring implementation
>
> Oops, previous email sent by mistake =E2=98=B9
>
> Hi, guys.
>
> I=E2=80=99ve submitted a PR, that refactors a cordova-lib API and impleme=
nts a
> PlatformApi polyfill (as discussed in
> https://na01.safelinks.protection.outlook.com/?url=3Dhttps%3a%2f%2fgithub=
.com%2fcordova%2fcordova-discuss%2fpull%2f12&data=3D01%7c01%7cv-vlkoti%4006=
4d.mgd.microsoft.com%7c9ff61bc6ceae43538c1008d2aa420cba%7c72f988bf86f141af9=
1ab2d7cd011db47%7c1&sdata=3DFd24ZInammyzOtVRlb2qBvHNxaohF9ms50j3dhIAkGs%3d
> )
> Please find it here:
> https://na01.safelinks.protection.outlook.com/?url=3Dhttps%3a%2f%2fgithub=
.com%2fapache%2fcordova-lib%2fpull%2f282.&data=3D01%7c01%7cv-vlkoti%40064d.=
mgd.microsoft.com%7c9ff61bc6ceae43538c1008d2aa420cba%7c72f988bf86f141af91ab=
2d7cd011db47%7c1&sdata=3DPzMbKu4Hb9Oe4NfkwBsIOZcwfcl31Oq%2fU%2fVdqBwlqgk%3d
> To test this you'll also need to check out corresponding PR to
> cordova-cli:
> https://na01.safelinks.protection.outlook.com/?url=3Dhttps%3a%2f%2fgithub=
.com%2fapache%2fcordova-cli%2fpull%2f220.&data=3D01%7c01%7cv-vlkoti%40064d.=
mgd.microsoft.com%7c9ff61bc6ceae43538c1008d2aa420cba%7c72f988bf86f141af91ab=
2d7cd011db47%7c1&sdata=3DIUZpWxvf1bu1x7Jb6UYLrdW10G6bMmvJ8Kl09%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 favo=
r
> of PlatformApiPoly/getPlatformApi
>   * make assets and js-modules installing/uninstalling through ActionStac=
k
>   * 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: dev-unsubscribe@cordova.apache.org
> For additional commands, e-mail: dev-help@cordova.apache.org
>

--047d7b67054d0566ba051e329d54--