Re: help with "npm test" for cordova-lib

Can someone help, perhaps Vladimir who wrote the tests can shed some light on how to fix this issue? On Mon, Apr 25, 2016 at 10:57 AM, Carlos Santana wrote: > It's jasmine test that is failing [1] > > Does this means that jasmine tests are not running in a continuously?

Re: Users mailing list

Or slack, but slack is only searchable back the last 10k messages. @purplecabbage risingj.com On Thu, Apr 28, 2016 at 5:36 PM, Shazron wrote: > Perhaps it's time for a users mailing list. No dev is expected to answer > questions on it. > > Example:

Re: [DISCUSS] Patch release for Cordova Plugin File

Re: cordova-lib release, I want to get this in: https://issues.apache.org/jira/browse/CB-11009 Not sure impact yet as long as unit tests pass On Wed, Apr 27, 2016 at 11:53 AM, Darryl Pogue wrote: > Not a plugin, but I'd appreciate a tools release (or at least a cordova-lib >

Users mailing list

Perhaps it's time for a users mailing list. No dev is expected to answer questions on it. Example: https://httpd.apache.org/userslist.html Right now we push people off to either: 1. Stack Overflow 2. PhoneGap Google Groups 3. Adobe PhoneGap Forums Thoughts?

Re: Grunt build and install taking too long after adding cordova plugin

This list is for the development of Cordova itself. For technical support questions, I recommend asking on Stack Overflow, or on the PhoneGap Google Group, you may get help more quickly. Although your issues sounds to be more about Oracle JET Hybrid Mobile than Cordova, are you sure you are in

[GitHub] cordova-plugin-wkwebview-engine pull request: CB-11074: Ensure set...

Github user shazron commented on the pull request: https://github.com/apache/cordova-plugin-wkwebview-engine/pull/8#issuecomment-215585938 I also commented on #7. To solve this problem, instead of having to instantiate WKWebView twice here, I think we can safely move all

[GitHub] cordova-plugin-wkwebview-engine pull request: Fixes CB-11074: WKWe...

Github user shazron commented on the pull request: https://github.com/apache/cordova-plugin-wkwebview-engine/pull/7#issuecomment-215584737 @ephemer not just settings (which as you said will probably not a frequent scenario), but setting the other delegates. updateWithInfo will be

[GitHub] cordova-android pull request: CB-11117: Use FileUpdater to optimiz...

Github user jasongin commented on a diff in the pull request: https://github.com/apache/cordova-android/pull/295#discussion_r61504885 --- Diff: bin/templates/cordova/lib/prepare.js --- @@ -84,29 +101,49 @@ function updateConfigFilesFrom(sourceConfig, configMunger, locations) {

[GitHub] cordova-android pull request: CB-11117: Use FileUpdater to optimiz...

Github user jasongin commented on a diff in the pull request: https://github.com/apache/cordova-android/pull/295#discussion_r61503079 --- Diff: bin/templates/cordova/lib/prepare.js --- @@ -26,26 +26,43 @@ var AndroidManifest = require('./AndroidManifest'); var xmlHelpers =

[GitHub] cordova-android pull request: CB-11117: Use FileUpdater to optimiz...

Github user vladimir-kotikov commented on a diff in the pull request: https://github.com/apache/cordova-android/pull/295#discussion_r61502339 --- Diff: bin/templates/cordova/lib/prepare.js --- @@ -26,26 +26,43 @@ var AndroidManifest = require('./AndroidManifest'); var

[GitHub] cordova-android pull request: CB-11138 Reuse PluginManager from co...

Github user jasongin commented on the pull request: https://github.com/apache/cordova-android/pull/301#issuecomment-215561101 :+1: --- 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

[GitHub] cordova-android pull request: CB-11117: Use FileUpdater to optimiz...

Github user vladimir-kotikov commented on a diff in the pull request: https://github.com/apache/cordova-android/pull/295#discussion_r61501081 --- Diff: bin/templates/cordova/lib/prepare.js --- @@ -26,26 +26,43 @@ var AndroidManifest = require('./AndroidManifest'); var

[GitHub] cordova-plugin-contacts pull request: Add fenced code blocks - wit...

GitHub user nikhilkh opened a pull request: https://github.com/apache/cordova-plugin-contacts/pull/124 Add fenced code blocks - with language hints You can merge this pull request into a Git repository by running: $ git pull https://github.com/nikhilkh/cordova-plugin-contacts

[GitHub] cordova-medic pull request: Adding configuration files to be used ...

Github user asfgit closed the pull request at: https://github.com/apache/cordova-medic/pull/91 --- 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

[GitHub] cordova-android pull request: CB-11138 Reuse PluginManager from co...

Github user vladimir-kotikov commented on the pull request: https://github.com/apache/cordova-android/pull/301#issuecomment-215537523 Sure, you can skip them safely. This is because we ship them as 'bundled' dependencies, since we don't run `npm install` after downloading the

[GitHub] cordova-android pull request: CB-11138 Reuse PluginManager from co...

Github user jasongin commented on the pull request: https://github.com/apache/cordova-android/pull/301#issuecomment-215533601 Can I ignore all the changes under node_modules? I don't understand why that is checked in to source control... --- If your project is set up for it, you can

[GitHub] cordova-docs pull request: CB-11169: Adding new repo links to cont...

Github user asfgit closed the pull request at: https://github.com/apache/cordova-docs/pull/588 --- 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

[GitHub] cordova-docs pull request: CB-11169: Adding new repo links to cont...

Github user rakatyal commented on the pull request: https://github.com/apache/cordova-docs/pull/588#issuecomment-215531427 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 have this feature

[GitHub] cordova-docs pull request: CB-11169: Adding new repo links to cont...

GitHub user riknoll opened a pull request: https://github.com/apache/cordova-docs/pull/588 CB-11169: Adding new repo links to contribute page @rakatyal You can merge this pull request into a Git repository by running: $ git pull https://github.com/MSOpenTech/cordova-docs

[GitHub] cordova-android pull request: CB-10096: Upgrading to Gradle Plugin...

Github user riknoll commented on the pull request: https://github.com/apache/cordova-android/pull/300#issuecomment-215524649 I tried it out a bit just now (without crosswalk). Java code changes refreshed fine for me. The webview gets restarted of course, but my java changes were

[GitHub] cordova-windows pull request: CB-11139 Use PluginManager from comm...

Github user nikhilkh commented on the pull request: https://github.com/apache/cordova-windows/pull/171#issuecomment-215523674 @jasongin Can you please help review? Related to apache/cordova-android#301 --- If your project is set up for it, you can reply to this email and have your

[GitHub] cordova-ios pull request: CB-11161 Reuse PluginManager from common...

Github user nikhilkh commented on the pull request: https://github.com/apache/cordova-ios/pull/219#issuecomment-215523478 @jasongin Can you please help review? Related to https://github.com/apache/cordova-android/pull/301 --- If your project is set up for it, you can reply to this

[GitHub] cordova-android pull request: CB-11138 Reuse PluginManager from co...

Github user nikhilkh commented on the pull request: https://github.com/apache/cordova-android/pull/301#issuecomment-215523214 @jasongin Can you please help review. --- 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

[GitHub] cordova-medic pull request: CB-11168: Fix the appName in cordova-m...

Github user asfgit closed the pull request at: https://github.com/apache/cordova-medic/pull/101 --- 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

[GitHub] cordova-android pull request: CB-10096: Upgrading to Gradle Plugin...

Github user infil00p commented on the pull request: https://github.com/apache/cordova-android/pull/300#issuecomment-215512407 @riknoll Did you test Instant Run on Android Studio? I don't expect this to work with Crosswalk. We need help from them getting Crosswalk to work with the

[GitHub] cordova-medic pull request: CB-11168: Fix the appName in cordova-m...

Github user omefire commented on the pull request: https://github.com/apache/cordova-medic/pull/101#issuecomment-215512291 Merging this without the suggested variable name changes, as we'll be moving away from cordova-medic soon anyway. --- If your project is set up for it, you can

[GitHub] cordova-android pull request: CB-10096: Upgrading to Gradle Plugin...

Github user riknoll commented on the pull request: https://github.com/apache/cordova-android/pull/300#issuecomment-215511375 I built/ran mobilespec with and without crosswalk and I didn't encounter any issues. One weird thing that did happen was that I wasn't able to increase the

[GitHub] cordova-medic pull request: CB-11168: Fix the appName in cordova-m...

Github user nikhilkh commented on the pull request: https://github.com/apache/cordova-medic/pull/101#issuecomment-215510394 This is a good find. Hopefully, this will fix all our issues with contacts permissions 👍 --- If your project is set up for it, you can reply to this email

Buildbot: buildslave cordova-windows-slave was lost

The Buildbot working for '' has noticed that the buildslave named cordova-windows-slave went away It last disconnected at Thu Apr 28 17:33:35 2016 (buildmaster-local time) The admin on record (as reported by BUILDSLAVE:info/admin) was 'Dmitry Blotsky '. Sincerely, The

[GitHub] cordova-plugin-contacts pull request: CB-11166 Added an Appium tes...

Github user alsorokin commented on a diff in the pull request: https://github.com/apache/cordova-plugin-contacts/pull/123#discussion_r61469757 --- Diff: appium-tests/common/common.spec.js --- @@ -114,14 +116,15 @@ describe('Contacts Android', function () { })

[GitHub] cordova-plugin-contacts pull request: CB-11166 Added an Appium tes...

Github user alsorokin commented on a diff in the pull request: https://github.com/apache/cordova-plugin-contacts/pull/123#discussion_r61469221 --- Diff: appium-tests/common/common.spec.js --- @@ -133,7 +136,7 @@ describe('Contacts Android', function () { function

[GitHub] cordova-plugin-contacts pull request: CB-11166 Added an Appium tes...

Github user riknoll commented on a diff in the pull request: https://github.com/apache/cordova-plugin-contacts/pull/123#discussion_r61468850 --- Diff: appium-tests/common/common.spec.js --- @@ -114,14 +116,15 @@ describe('Contacts Android', function () { })

[GitHub] cordova-medic pull request: CB-11168: Fix the appName in cordova-m...

Github user omefire commented on the pull request: https://github.com/apache/cordova-medic/pull/101#issuecomment-215503182 @sarangan12 a quick search through the repo yields only 4 files with less than 10 instances that would need to be updated. I think it is worth the effort, :)

[GitHub] cordova-medic pull request: CB-11168: Fix the appName in cordova-m...

Github user sarangan12 commented on the pull request: https://github.com/apache/cordova-medic/pull/101#issuecomment-215502169 @omefire Agreed. But that would involve additiional code changes which are not related directly to this issue. You can create a seperate JIRA for it. Also, I

[GitHub] cordova-plugin-contacts pull request: CB-11166 Added an Appium tes...

Github user alsorokin commented on a diff in the pull request: https://github.com/apache/cordova-plugin-contacts/pull/123#discussion_r61468116 --- Diff: appium-tests/common/common.spec.js --- @@ -114,14 +116,15 @@ describe('Contacts Android', function () { })

[GitHub] cordova-plugin-contacts pull request: CB-11166 Added an Appium tes...

Github user riknoll commented on a diff in the pull request: https://github.com/apache/cordova-plugin-contacts/pull/123#discussion_r61467646 --- Diff: appium-tests/common/common.spec.js --- @@ -114,14 +116,15 @@ describe('Contacts Android', function () { })

[GitHub] cordova-medic pull request: CB-11168: Fix the appName in cordova-m...

Github user omefire commented on the pull request: https://github.com/apache/cordova-medic/pull/101#issuecomment-215501596 Hmm, we've got a case of confusing naming here. I think we should rename ```argv.app``` to ```argv.appName```. That will make the intent clearer and prevent such

[GitHub] cordova-plugin-contacts pull request: CB-11166 Added an Appium tes...

Github user riknoll commented on a diff in the pull request: https://github.com/apache/cordova-plugin-contacts/pull/123#discussion_r61467710 --- Diff: appium-tests/common/common.spec.js --- @@ -133,7 +136,7 @@ describe('Contacts Android', function () { function

[GitHub] cordova-medic pull request: CB-11168: Fix the appName in cordova-m...

GitHub user sarangan12 opened a pull request: https://github.com/apache/cordova-medic/pull/101 CB-11168: Fix the appName in cordova-medic The appName is just set to incorrect value. The appName must be 'org.apache.mobilespec'. But, when it is picked from command line arguments,

Grunt build and install taking too long after adding cordova plugin

In order to support push notification in my hybrid application, I added phonegap-plugin-push. But after adding this plugin, the application build process takes more than 20 minutes! (10 for build and 10 for serve) Here are the steps: 1.HYPERLINK

[GitHub] cordova-android pull request: CB-11117: Use FileUpdater to optimiz...

Github user jasongin commented on a diff in the pull request: https://github.com/apache/cordova-android/pull/295#discussion_r61462710 --- Diff: bin/templates/cordova/lib/prepare.js --- @@ -26,26 +26,43 @@ var AndroidManifest = require('./AndroidManifest'); var xmlHelpers =

[GitHub] cordova-android pull request: CB-11117: Use FileUpdater to optimiz...

Github user jasongin commented on a diff in the pull request: https://github.com/apache/cordova-android/pull/295#discussion_r61461619 --- Diff: bin/templates/cordova/lib/prepare.js --- @@ -26,26 +26,43 @@ var AndroidManifest = require('./AndroidManifest'); var xmlHelpers =

[GitHub] cordova-plugin-wkwebview-engine pull request: CB-11074: Ensure set...

GitHub user ephemer opened a pull request: https://github.com/apache/cordova-plugin-wkwebview-engine/pull/8 CB-11074: Ensure settings from config.xml are taken into consideration #7 also addresses this issue but in my tests didn't work. This pull request presents a

[GitHub] cordova-android pull request: CB-11163 Android Builds are failing ...

Github user daserge commented on the pull request: https://github.com/apache/cordova-android/pull/302#issuecomment-215487630 Reverted https://github.com/apache/cordova-android/commit/abcaee24f9bd0e680f8d06ced5e0d363d63cffb7 to unblock the CI. Reopened the

[GitHub] cordova-android pull request: CB-11163 Android Builds are failing ...

Github user nikhilkh commented on the pull request: https://github.com/apache/cordova-android/pull/302#issuecomment-215486540 Can you explain what was the issue here and why this fixes it? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cordova-android pull request: CB-11163 Android Builds are failing ...

Github user asfgit closed the pull request at: https://github.com/apache/cordova-android/pull/302 --- 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

[GitHub] cordova-lib pull request: CB-11023 Add attribute through config-fi...

Github user codecov-io commented on the pull request: https://github.com/apache/cordova-lib/pull/432#issuecomment-215478064 ## [Current coverage][cc-pull] is **79.60%** > Merging [#432][cc-pull] into [master][cc-base-branch] will increase coverage by **-0.49%** ```diff

[GitHub] cordova-plugin-contacts pull request: CB-11166 Added an Appium tes...

GitHub user alsorokin opened a pull request: https://github.com/apache/cordova-plugin-contacts/pull/123 CB-11166 Added an Appium tests to check for contact's birthday https://issues.apache.org/jira/browse/CB-11166 You can merge this pull request into a Git repository by running:

[GitHub] cordova-plugin-wkwebview-engine pull request: Fixes CB-11074: WKWe...

Github user ephemer commented on the pull request: https://github.com/apache/cordova-plugin-wkwebview-engine/pull/7#issuecomment-215453411 As I commented in the base repo, this PR doesn't do what it says it does yet. @shazron is it normal to change the settings after the

[GitHub] cordova-android pull request: CB-11163 Android Builds are failing ...

Github user codecov-io commented on the pull request: https://github.com/apache/cordova-android/pull/302#issuecomment-215448214 ## [Current coverage][cc-pull] is **100%** > Merging [#302][cc-pull] into [master][cc-base-branch] will not change coverage ```diff @@

[GitHub] cordova-android pull request: CB-11163 Android Builds are failing ...

GitHub user daserge opened a pull request: https://github.com/apache/cordova-android/pull/302 CB-11163 Android Builds are failing in CI CB-11133 Handle android emulator start failure (reverted from commit abcaee24f9bd0e680f8d06ced5e0d363d63cffb7) You can merge this pull request

[GitHub] cordova-cli pull request: CB-11100 Update dependencies to avoid in...

GitHub user vladimir-kotikov opened a pull request: https://github.com/apache/cordova-cli/pull/246 CB-11100 Update dependencies to avoid installation warnings You can merge this pull request into a Git repository by running: $ git pull

[GitHub] cordova-ios pull request: CB-11161 Reuse PluginManager from common...

GitHub user vladimir-kotikov opened a pull request: https://github.com/apache/cordova-ios/pull/219 CB-11161 Reuse PluginManager from common to add/rm plugins A change to use `PluginManager` class to manage plugin installation/uninstallation instead of custom logic JIRA:

[GitHub] cordova-android pull request: CB-11138 Reuse PluginManager from co...

GitHub user vladimir-kotikov opened a pull request: https://github.com/apache/cordova-android/pull/301 CB-11138 Reuse PluginManager from common to add/rm plugins A change to use `PluginManager` class to manage plugin installation/uninstallation instead of custom logic

[GitHub] cordova-windows pull request: CB-11139 Use PluginManager from comm...

Github user codecov-io commented on the pull request: https://github.com/apache/cordova-windows/pull/171#issuecomment-215373720 ## [Current coverage][cc-pull] is **100%** > Merging [#171][cc-pull] into [master][cc-base-branch] will not change coverage ```diff @@

[GitHub] cordova-windows pull request: CB-11139 Use PluginManager from comm...

GitHub user vladimir-kotikov opened a pull request: https://github.com/apache/cordova-windows/pull/171 CB-11139 Use PluginManager from common to install/uninstall plugins This PR removes a lot of the code, related to plugin installation in favor of unified `PluginManager` class,

[GitHub] cordova-android pull request: CB-11117: Use FileUpdater to optimiz...

Github user vladimir-kotikov commented on a diff in the pull request: https://github.com/apache/cordova-android/pull/295#discussion_r61393298 --- Diff: bin/templates/cordova/lib/prepare.js --- @@ -295,25 +344,47 @@ function handleIcons(projectConfig, platformRoot) {

[GitHub] cordova-android pull request: CB-11117: Use FileUpdater to optimiz...

Github user vladimir-kotikov commented on a diff in the pull request: https://github.com/apache/cordova-android/pull/295#discussion_r61392785 --- Diff: bin/templates/cordova/lib/prepare.js --- @@ -26,26 +26,43 @@ var AndroidManifest = require('./AndroidManifest'); var

[GitHub] cordova-android pull request: CB-11117: Use FileUpdater to optimiz...

Github user vladimir-kotikov commented on a diff in the pull request: https://github.com/apache/cordova-android/pull/295#discussion_r61391425 --- Diff: bin/templates/cordova/lib/prepare.js --- @@ -26,26 +26,43 @@ var AndroidManifest = require('./AndroidManifest'); var

[GitHub] cordova-android pull request: CB-11117: Use FileUpdater to optimiz...

Github user vladimir-kotikov commented on a diff in the pull request: https://github.com/apache/cordova-android/pull/295#discussion_r61388176 --- Diff: bin/templates/cordova/lib/prepare.js --- @@ -84,29 +101,49 @@ function updateConfigFilesFrom(sourceConfig, configMunger,

[GitHub] cordova-android pull request: CB-11117: Use FileUpdater to optimiz...

Github user vladimir-kotikov commented on a diff in the pull request: https://github.com/apache/cordova-android/pull/295#discussion_r61388100 --- Diff: bin/templates/cordova/lib/prepare.js --- @@ -84,29 +101,49 @@ function updateConfigFilesFrom(sourceConfig, configMunger,

[GitHub] cordova-plugin-camera pull request: CB-11162 Appium tests: retry s...

Github user asfgit closed the pull request at: https://github.com/apache/cordova-plugin-camera/pull/208 --- 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

[GitHub] cordova-android pull request: CB-11117: Use FileUpdater to optimiz...

Github user vladimir-kotikov commented on a diff in the pull request: https://github.com/apache/cordova-android/pull/295#discussion_r61383879 --- Diff: bin/templates/cordova/lib/prepare.js --- @@ -26,26 +26,43 @@ var AndroidManifest = require('./AndroidManifest'); var

[GitHub] cordova-android pull request: CB-11117: Use FileUpdater to optimiz...

Github user vladimir-kotikov commented on a diff in the pull request: https://github.com/apache/cordova-android/pull/295#discussion_r61383391 --- Diff: bin/templates/cordova/lib/prepare.js --- @@ -26,26 +26,43 @@ var AndroidManifest = require('./AndroidManifest'); var