Merge pull request #277 from brodybits/quick-ci-fixes

Quick CI fixes

.appveyor.yml

@@ -21,8 +21,10 @@ install:
   - node --version
   - npm install -g cordova-paramedic@https://github.com/apache/cordova-paramedic.git
   - npm install -g cordova
+  - npm install
 
 build: off
 
 test_script:
+  - npm run eslint
   - cordova-paramedic --config pr\%PLATFORM% --plugin . %JUST_BUILD%

.travis.yml

@@ -1,11 +1,14 @@
 sudo: false
+
 addons:
   jwt:
     secure: TZ88IEvAw1bsWPWxvDzXdpi2NK0i3PN4hG15+vDpIt6wXGVPknjxuXWJeLj7TqBpAIvP7XDfS8ZvHVPLe7fe8oOchZPLuiDw9VVIk6cnHjE6wpoavdGc/1mDJ3Bi4PDcHwRUr5ng5spYQqqlTwcECkH/q7iPgudiFM6rlOlGRyA=
+
 env:
   global:
   - SAUCE_USERNAME=snay
   - TRAVIS_NODE_VERSION="4.2"
+
 matrix:
   include:
   - env: PLATFORM=browser-chrome
@@ -16,10 +19,11 @@ matrix:
     os: linux
     language: node_js
     node_js: '4.2'
-  - env: PLATFORM=browser-safari
-    os: linux
-    language: node_js
-    node_js: '4.2'
+  # TBD SKIP for now:
+  # - env: PLATFORM=browser-safari
+  #   os: linux
+  #   language: node_js
+  #   node_js: '4.2'
   - env: PLATFORM=browser-edge
     os: linux
     language: node_js
@@ -29,11 +33,15 @@ matrix:
     osx_image: xcode7.3
     language: node_js
     node_js: '4.2'
-  - env: PLATFORM=ios-10.0
-    os: osx
-    osx_image: xcode7.3
-    language: node_js
-    node_js: '4.2'
+  # TBD SKIP for now:
+  # - env: PLATFORM=ios-10.0
+  #   os: osx
+  #   osx_image: xcode7.3
+  #   language: node_js
+  #   node_js: '4.2'
+  # FUTURE TBD:
+  # - env: PLATFORM=ios-11.4
+  #   ...
   - env: PLATFORM=android-4.4
     os: linux
     language: android
@@ -44,36 +52,41 @@ matrix:
       - platform-tools
       - tools
       - build-tools-26.0.2
-  - env: PLATFORM=android-5.1
-    os: linux
-    language: android
-    jdk: oraclejdk8
-    android:
-      components:
-      - tools
-      - platform-tools
-      - tools
-      - build-tools-26.0.2
-  - env: PLATFORM=android-6.0
-    os: linux
-    language: android
-    jdk: oraclejdk8
-    android:
-      components:
-      - tools
-      - platform-tools
-      - tools
-      - build-tools-26.0.2
-  - env: PLATFORM=android-7.0
-    os: linux
-    language: android
-    jdk: oraclejdk8
-    android:
-      components:
-      - tools
-      - platform-tools
-      - tools
-      - build-tools-26.0.2
+  # TBD SKIP for now:
+  # - env: PLATFORM=android-5.1
+  #   os: linux
+  #   language: android
+  #   jdk: oraclejdk8
+  #   android:
+  #     components:
+  #     - tools
+  #     - platform-tools
+  #     - tools
+  #     - build-tools-26.0.2
+  # - env: PLATFORM=android-6.0
+  #   os: linux
+  #   language: android
+  #   jdk: oraclejdk8
+  #   android:
+  #     components:
+  #     - tools
+  #     - platform-tools
+  #     - tools
+  #     - build-tools-26.0.2
+  # - env: PLATFORM=android-7.0
+  #   os: linux
+  #   language: android
+  #   jdk: oraclejdk8
+  #   android:
+  #     components:
+  #     - tools
+  #     - platform-tools
+  #     - tools
+  #     - build-tools-26.0.2
+  # FUTURE TBD:
+  # - env: PLATFORM=android-8.?
+  #   ...
+
 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
@@ -86,8 +99,10 @@ before_install:
 - git clone https://github.com/apache/cordova-paramedic /tmp/paramedic && pushd /tmp/paramedic
   && npm install && popd
 - npm install -g cordova
+
 install:
 - npm install
+
 script:
 - npm test
 - node /tmp/paramedic/main.js --config pr/$PLATFORM --plugin $(pwd) --shouldUseSauce

README.md

@@ -24,6 +24,11 @@ description: Open an in-app browser window.
 |AppVeyor|Travis CI|
 |:-:|:-:|
 |[![Build status](https://ci.appveyor.com/api/projects/status/github/apache/cordova-plugin-inappbrowser?branch=master)](https://ci.appveyor.com/project/ApacheSoftwareFoundation/cordova-plugin-inappbrowser)|[![Build Status](https://travis-ci.org/apache/cordova-plugin-inappbrowser.svg?branch=master)](https://travis-ci.org/apache/cordova-plugin-inappbrowser)|
+|npm install|npm install|
+|eslint|eslint|
+||browser (chrome, firefox, edge)|
+||iOS (9.3) on Xcode 7.3|
+||Android (4.4)|
 
 # cordova-plugin-inappbrowser