You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cordova.apache.org by ja...@apache.org on 2018/11/24 17:37:53 UTC

[cordova-paramedic] branch janpio-misc updated (0494735 -> 8efb65d)

This is an automated email from the ASF dual-hosted git repository.

janpio pushed a change to branch janpio-misc
in repository https://gitbox.apache.org/repos/asf/cordova-paramedic.git.


 discard 0494735  nicer log message
     add 86101e4  Support physical Android devices (#24)
     add dfed2c7  Fix inappbrowser on Android 7 (#50)
     add 2b24b0a  most rudimentary package.json for plugins and plugin tests possible
     add ac87c1c  nobody needs wp8 any more, but browser would be useful
     add da22d52  Merge pull request #35 from apache/janpio-replace_wp8_with_browser
     add 3fe8555  remove wp8 ref
     add 4dd8190  Remove wp8 ref
     add 2b3e03f  [CB-12210] require a string when parsing the plugin argument. (#19)
     new 4c17218  nicer log message
     new 26531a9  log all executed command explicitly
     new 8efb65d  some comments what the script is doing

This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version.  This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:

 * -- * -- B -- O -- O -- O   (0494735)
            \
             N -- N -- N   refs/heads/janpio-misc (8efb65d)

You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.

Any revisions marked "omit" are not gone; other references still
refer to them.  Any revisions marked "discard" are gone forever.

The 3 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 README.md                     | 11 +++++++++--
 lib/ParamedicTargetChooser.js |  8 ++++++++
 lib/paramedic.js              | 12 ++++++++++--
 lib/utils/execWrapper.js      |  1 +
 main.js                       |  6 ++++--
 package.json                  |  2 +-
 6 files changed, 33 insertions(+), 7 deletions(-)


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org
For additional commands, e-mail: commits-help@cordova.apache.org


[cordova-paramedic] 02/03: log all executed command explicitly

Posted by ja...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

janpio pushed a commit to branch janpio-misc
in repository https://gitbox.apache.org/repos/asf/cordova-paramedic.git

commit 26531a9fdf2d70015f5d85e009199760c0544797
Author: Jan Piotrowski <pi...@gmail.com>
AuthorDate: Sat Nov 24 18:31:04 2018 +0100

    log all executed command explicitly
---
 lib/utils/execWrapper.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/lib/utils/execWrapper.js b/lib/utils/execWrapper.js
index e00be10..ab2f95a 100644
--- a/lib/utils/execWrapper.js
+++ b/lib/utils/execWrapper.js
@@ -22,6 +22,7 @@ var Q = require('q');
 var verbose;
 
 function exec(cmd, onFinish, onData) {
+    console.log('$ ' + cmd)
     if (onFinish instanceof Function || onFinish === null) {
         var result = shelljs.exec(cmd, {async: true, silent: !verbose}, onFinish);
 


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org
For additional commands, e-mail: commits-help@cordova.apache.org


[cordova-paramedic] 01/03: nicer log message

Posted by ja...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

janpio pushed a commit to branch janpio-misc
in repository https://gitbox.apache.org/repos/asf/cordova-paramedic.git

commit 4c1721876f43ba5d9506d79a6fed940b1ad34ea3
Author: Jan Piotrowski <pi...@gmail.com>
AuthorDate: Tue Oct 2 20:33:20 2018 +0200

    nicer log message
---
 lib/paramedic.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/paramedic.js b/lib/paramedic.js
index a386bb7..016de98 100644
--- a/lib/paramedic.js
+++ b/lib/paramedic.js
@@ -184,7 +184,7 @@ ParamedicRunner.prototype.installPlatform = function () {
     var self = this;
     var platform = this.config.getPlatform();
     var platformId = this.config.getPlatformId();
-    logger.info('cordova-paramedic: adding platform ' + platform + "(" + util.PARAMEDIC_COMMON_CLI_ARGS + util.PARAMEDIC_PLATFORM_ADD_ARGS + ")");
+    logger.info('cordova-paramedic: adding platform ' + platform + " (with: " + util.PARAMEDIC_COMMON_CLI_ARGS + util.PARAMEDIC_PLATFORM_ADD_ARGS + ")");
 
     return execPromise(this.config.getCli() + ' platform add ' + platform + util.PARAMEDIC_COMMON_CLI_ARGS + util.PARAMEDIC_PLATFORM_ADD_ARGS)
     .then(function () {


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org
For additional commands, e-mail: commits-help@cordova.apache.org


[cordova-paramedic] 03/03: some comments what the script is doing

Posted by ja...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

janpio pushed a commit to branch janpio-misc
in repository https://gitbox.apache.org/repos/asf/cordova-paramedic.git

commit 8efb65db8dad5cd4545d10c57f68b49693a31ba2
Author: Jan Piotrowski <pi...@gmail.com>
AuthorDate: Sat Nov 24 18:31:59 2018 +0100

    some comments what the script is doing
---
 lib/paramedic.js | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/lib/paramedic.js b/lib/paramedic.js
index 016de98..9bc7a26 100644
--- a/lib/paramedic.js
+++ b/lib/paramedic.js
@@ -70,18 +70,21 @@ ParamedicRunner.prototype.run = function () {
     self.checkConfig();
 
     return Q().then(function () {
+        // create project and prepare (install plugins, setup test startpage, install platform, check platform requirements)
         self.createTempProject();
         shell.pushd(self.tempFolder.name);
         return self.prepareProjectToRunTests();
     })
     .then(function () {
         if (self.config.runMainTests()) {
+            // start server
             var noListener = (self.config.getPlatformId() === util.BROWSER) && self.config.shouldUseSauce();
             return Server.startServer(self.config.getPorts(), self.config.getExternalServerUrl(), self.config.getUseTunnel(), noListener);
         }
     })
     .then(function (server) {
         if (self.config.runMainTests()) {
+            // configure server usage
             self.server = server;
 
             self.injectReporters();
@@ -92,6 +95,7 @@ ParamedicRunner.prototype.run = function () {
 
             logger.normal('Start running tests at ' + (new Date()).toLocaleTimeString());
         }
+        // run tests
         return self.runTests();
     })
     .timeout(self.config.getTimeout(), 'Timed out after waiting for ' + self.config.getTimeout() + ' ms.')
@@ -101,6 +105,7 @@ ParamedicRunner.prototype.run = function () {
         // if we do --justbuild  or run on sauce,
         // we should NOT do actions below
         if (self.config.getAction() !== 'build' && !self.config.shouldUseSauce()) {
+            // collect logs and uninstall app
             self.collectDeviceLogs();
             return self.uninstallApp()
                 .fail(function () {


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org
For additional commands, e-mail: commits-help@cordova.apache.org