RE: [GitHub] cordova-lib pull request: Adds implementation for PlatformApiPoly ...

2015-08-26 Thread Vladimir Kotikov (Akvelon)
Sure, Jesse. Best regards, Vladimir. -Original Message- From: Jesse [mailto:purplecabb...@gmail.com] Sent: Wednesday, August 26, 2015 3:58 AM To: dev@cordova.apache.org Subject: Re: [GitHub] cordova-lib pull request: Adds implementation for PlatformApiPoly ... This looks great, but I

[GitHub] cordova-lib pull request: Adds implementation for PlatformApiPoly ...

2015-08-25 Thread stevengill
Github user stevengill commented on the pull request: https://github.com/apache/cordova-lib/pull/282#issuecomment-134780081 Looks great Vlad! I'm fine with you merging this in and us spending some time testing it on master to make sure it works with all of our platforms. --- If

Re: [GitHub] cordova-lib pull request: Adds implementation for PlatformApiPoly ...

2015-08-25 Thread Jesse
This looks great, but I would like to spend a little more time before we push it to master. Can we hold off until the end of the week? My team is hiring! @purplecabbage risingj.com On Tue, Aug 25, 2015 at 5:46 PM, stevengill g...@git.apache.org wrote: Github user stevengill commented on the

[GitHub] cordova-lib pull request: Adds implementation for PlatformApiPoly ...

2015-08-24 Thread vladimir-kotikov
Github user vladimir-kotikov commented on the pull request: https://github.com/apache/cordova-lib/pull/282#issuecomment-134241500 Thanks for advice, Mark. I have some plans about tests update and definitely will try fixtures-based approach. --- If your project is set up for it, you

[GitHub] cordova-lib pull request: Adds implementation for PlatformApiPoly ...

2015-08-24 Thread vladimir-kotikov
Github user vladimir-kotikov commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/282#discussion_r37758129 --- Diff: cordova-lib/src/plugman/util/config-changes.js --- @@ -168,22 +167,18 @@ function remove_plugin_changes(plugin_name, plugin_id,

[GitHub] cordova-lib pull request: Adds implementation for PlatformApiPoly ...

2015-08-21 Thread vladimir-kotikov
GitHub user vladimir-kotikov opened a pull request: https://github.com/apache/cordova-lib/pull/282 Adds implementation for PlatformApiPoly class according to PlatformApi spec This implements PlatformApiPoly class according to PlatformApi spec, which allows to: * create/update