[GitHub] janpio commented on issue #37: [Plugins] Improve plugin tests

[GitBox <gi...@apache.org>]

janpio commented on issue #37: [Plugins] Improve plugin tests
URL: https://github.com/apache/cordova/issues/37#issuecomment-425859315
 
 
   `.travis.yml` has this bit:
   
   ```
   before_install:
   - rm -rf ~/.nvm && git clone https://github.com/creationix/nvm.git ~/.nvm && (cd ~/.nvm && git checkout `git describe --abbrev=0 --tags`) && source ~/.nvm/nvm.sh && nvm install $TRAVIS_NODE_VERSION
   - node --version
   - if [[ "$PLATFORM" =~ android ]]; then gradle --version; fi
   - if [[ "$PLATFORM" =~ ios ]]; then npm install -g ios-deploy; fi
   - if [[ "$PLATFORM" =~ android ]]; then echo y | android update sdk -u --filter android-22,android-23,android-24,android-25,android-26,android-27;
     fi
   - git clone https://github.com/apache/cordova-paramedic /tmp/paramedic && pushd
     /tmp/paramedic && npm install && popd
   - npm install -g cordova
   ```
   (or similar, as configs can be different)
   
   This should:
   1. not have those confusing line breaks
   2. get the `android-22,android-23,android-24,android-25,android-26,android-27` string from a variable so that it is not that easy to overlook
   3. probably something like nvm is available on travis by default?
   4. not switch from android, then to ios, then again android

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

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