Skip to content

Commit

Permalink
Auto merge of #6536 - kellyselden:travis, r=rwjblue
Browse files Browse the repository at this point in the history
fix phantom use on travis

I noticed an issue where phantom 1.9 was being used even though we were expecting 2.1 https://travis-ci.org/ember-cli/ember-addon-output/jobs/180486782#L447-L468

turns out we needed to install it globally. Here it is in action: https://travis-ci.org/kellyselden/ember-datetimepicker/jobs/181879916#L472

I targeted release branch because this seemed like a pretty silly/low risk bug fix that we should get to people soon.
  • Loading branch information
homu committed Dec 7, 2016
2 parents 8f78a26 + b1e2099 commit f9d86df
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 6 deletions.
5 changes: 2 additions & 3 deletions blueprints/addon/files/.travis.yml
Expand Up @@ -25,10 +25,9 @@ matrix:

before_install:
- npm config set spin false
- npm install -g bower
- npm install -g bower phantomjs-prebuilt
- bower --version
- npm install phantomjs-prebuilt
- node_modules/phantomjs-prebuilt/bin/phantomjs --version
- phantomjs --version

install:
- npm install
Expand Down
5 changes: 2 additions & 3 deletions blueprints/app/files/.travis.yml
Expand Up @@ -12,10 +12,9 @@ cache:

before_install:
- npm config set spin false
- npm install -g bower
- npm install -g bower phantomjs-prebuilt
- bower --version
- npm install phantomjs-prebuilt
- node_modules/phantomjs-prebuilt/bin/phantomjs --version
- phantomjs --version

install:
- npm install
Expand Down

0 comments on commit f9d86df

Please sign in to comment.