[GitHub] cordova-plugin-geolocation pull request: CB-10636 Add JSHint for p...

2016-02-27 Thread vladimir-kotikov
Github user vladimir-kotikov commented on the pull request: https://github.com/apache/cordova-plugin-geolocation/pull/72#issuecomment-189606016 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does no

[GitHub] cordova-lib pull request: CB-10314 avoid fetching plugins when old...

2016-02-27 Thread vladimir-kotikov
Github user vladimir-kotikov commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/396#discussion_r54331174 --- Diff: cordova-lib/src/plugman/fetch.js --- @@ -135,13 +135,32 @@ function fetchPlugin(plugin_src, plugins_dir, options) { v

[GitHub] cordova-lib pull request: CB-10314 avoid fetching plugins when old...

2016-02-27 Thread vladimir-kotikov
Github user vladimir-kotikov commented on the pull request: https://github.com/apache/cordova-lib/pull/396#issuecomment-189612511 @bso-intel, thanks. This looks good apart from the logic duplication in `plugman/install` and `plugman/fetch`. However, this looks like is out of the scope

[GitHub] cordova-lib pull request: CB-10462 Get rid of npmconf in favor of ...

2016-02-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-lib/pull/401 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] cordova-plugin-statusbar pull request: CB-10636 CB-10683 Add JSHin...

2016-02-27 Thread vladimir-kotikov
Github user vladimir-kotikov commented on the pull request: https://github.com/apache/cordova-plugin-statusbar/pull/49#issuecomment-189613712 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] cordova-plugin-wkwebview-engine pull request: CB-10636 Add JSHint ...

2016-02-27 Thread vladimir-kotikov
Github user vladimir-kotikov commented on the pull request: https://github.com/apache/cordova-plugin-wkwebview-engine/pull/6#issuecomment-189613962 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project doe

[GitHub] cordova-plugin-file-transfer pull request: CB-10636 Add JSHint for...

2016-02-27 Thread vladimir-kotikov
Github user vladimir-kotikov commented on a diff in the pull request: https://github.com/apache/cordova-plugin-file-transfer/pull/127#discussion_r54331337 --- Diff: .jscsrc --- @@ -1,7 +1,7 @@ { "disallowMixedSpacesAndTabs": true, "disallowTrailingWhitespace"

[GitHub] cordova-plugin-file-transfer pull request: CB-10636 Add JSHint for...

2016-02-27 Thread vladimir-kotikov
Github user vladimir-kotikov commented on a diff in the pull request: https://github.com/apache/cordova-plugin-file-transfer/pull/127#discussion_r54331382 --- Diff: package.json --- @@ -20,8 +20,8 @@ }, "scripts": { "test": "npm run lint && npm run style",

[GitHub] cordova-plugin-file-transfer pull request: CB-10636 Add JSHint for...

2016-02-27 Thread vladimir-kotikov
Github user vladimir-kotikov commented on the pull request: https://github.com/apache/cordova-plugin-file-transfer/pull/127#issuecomment-189614910 LGTm --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

[GitHub] cordova-plugin-battery-status pull request: CB-10636 Add JSHint fo...

2016-02-27 Thread vladimir-kotikov
Github user vladimir-kotikov commented on the pull request: https://github.com/apache/cordova-plugin-battery-status/pull/32#issuecomment-189614969 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

[GitHub] cordova-plugin-console pull request: CB-10636 Add JSHint for plugi...

2016-02-27 Thread vladimir-kotikov
Github user vladimir-kotikov commented on the pull request: https://github.com/apache/cordova-plugin-console/pull/16#issuecomment-189614983 LGTm --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not ha

[GitHub] cordova-plugin-device-motion pull request: CB-10636 Add JSHint for...

2016-02-27 Thread vladimir-kotikov
Github user vladimir-kotikov commented on the pull request: https://github.com/apache/cordova-plugin-device-motion/pull/41#issuecomment-189615176 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

[GitHub] cordova-plugin-device-orientation pull request: CB-10636 Add JSHin...

2016-02-27 Thread vladimir-kotikov
Github user vladimir-kotikov commented on the pull request: https://github.com/apache/cordova-plugin-device-orientation/pull/25#issuecomment-189615205 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] cordova-plugin-dialogs pull request: CB-10636 Add JSHint for plugi...

2016-02-27 Thread vladimir-kotikov
Github user vladimir-kotikov commented on the pull request: https://github.com/apache/cordova-plugin-dialogs/pull/65#issuecomment-189615257 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not ha

[GitHub] cordova-plugin-geolocation pull request: CB-10636 Add JSHint for p...

2016-02-27 Thread vladimir-kotikov
Github user vladimir-kotikov commented on the pull request: https://github.com/apache/cordova-plugin-geolocation/pull/72#issuecomment-189615537 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does no

[GitHub] cordova-plugin-file-transfer pull request: Allow to pass an ArrayB...

2016-02-27 Thread tuvistavie
GitHub user tuvistavie opened a pull request: https://github.com/apache/cordova-plugin-file-transfer/pull/128 Allow to pass an ArrayBuffer to the upload function when working in the browser. Hi, I have been working with this plugin to create an app compatible with both

[GitHub] cordova-lib pull request: CB-10314 avoid fetching plugins when old...

2016-02-27 Thread bso-intel
Github user bso-intel commented on the pull request: https://github.com/apache/cordova-lib/pull/396#issuecomment-189672357 Yes, please merge this for me. I don't have committer privilege, which I wish to get. :) Thanks, Vladimir. --- If your project is set up for it, you can r

[GitHub] cordova-lib pull request: CB-10314 avoid fetching plugins when old...

2016-02-27 Thread bso-intel
Github user bso-intel commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/396#discussion_r54334968 --- Diff: cordova-lib/src/plugman/fetch.js --- @@ -135,13 +135,32 @@ function fetchPlugin(plugin_src, plugins_dir, options) { var spli

[GitHub] cordova-plugin-camera pull request: Android FileHelper modificatio...

2016-02-27 Thread dlogo
Github user dlogo commented on the pull request: https://github.com/apache/cordova-plugin-camera/pull/149#issuecomment-189703384 @dallas sorry for answering late, I have checked the differences between the PR and the base and I think that merge should be done if the PR is checked by

[GitHub] cordova-ios pull request: CB-10673 fixed conflicting plugin instal...

2016-02-27 Thread bso-intel
GitHub user bso-intel opened a pull request: https://github.com/apache/cordova-ios/pull/199 CB-10673 fixed conflicting plugin install issue with overlapped tag The problem is that copyNewFile is too strict for tag. You will never know which two plugins will write the library to

[GitHub] cordova-android pull request: CB-10673 fixed conflicting plugin in...

2016-02-27 Thread bso-intel
GitHub user bso-intel opened a pull request: https://github.com/apache/cordova-android/pull/264 CB-10673 fixed conflicting plugin install issue with overlapped tag The problem is that copyNewFile is too strict for tag. You will never know which two plugins will write the library

[GitHub] cordova-windows pull request: CB-10673 CB-10673 fixed conflicting ...

2016-02-27 Thread bso-intel
GitHub user bso-intel opened a pull request: https://github.com/apache/cordova-windows/pull/154 CB-10673 CB-10673 fixed conflicting plugin install issue with overlapped The problem is that copyNewFile is too strict for tag. You will never know which two plugins will write the