Merge pull request #277 from brodybits/quick-ci-fixes
Quick CI fixes
diff --git a/.appveyor.yml b/.appveyor.yml
index a7b2426..80e8353 100644
--- a/.appveyor.yml
+++ b/.appveyor.yml
@@ -21,8 +21,10 @@
- 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%
diff --git a/.travis.yml b/.travis.yml
index 6098fe3..00def86 100644
--- a/.travis.yml
+++ b/.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 @@
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 @@
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 @@
- 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 @@
- 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
diff --git a/README.md b/README.md
index 26ed518..eade58a 100644
--- a/README.md
+++ b/README.md
@@ -24,6 +24,11 @@
|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