Bump test specs #1623

Merged
merged 5 commits into from Apr 25, 2016

Projects

None yet

5 participants

@samccone
Member
samccone commented Apr 24, 2016 edited

* Switched to tabs in .travis.yml

  • Updated chrome version
  • Cached node_modules
  • Updated selenium version
  • remove unneeded chrome driver DL
  • update Drool
  • Remove direct call to webdriver log
@mention-bot

By analyzing the blame information on this pull request, we identified @passy, @sindresorhus and @ipetropolsky to be potential reviewers

@samccone
Member

odd, travis is not picking this up at all...

@sindresorhus
Member

Switched to tabs in .travis.yml

You can't indent with tabs in YAML.

samccone added some commits Apr 24, 2016
@samccone samccone tests: cache node_modules
Move chrome to snapshots
009c135
@samccone samccone tests: Bump selenium version 2a204e7
@samccone samccone tests: Bump drool to v0.4.0
69f2556
@samccone samccone tests: Remove chromedriver download 0a17d95
@samccone samccone test: Remove direct logs call.
8ec1db0
@tbranyen
tbranyen commented Apr 24, 2016 edited

Nice will this fix the issues I was seeing in #1621?

@samccone
Member

@tbranyen this will be a good first step :)

@tbranyen

Awesome I'll follow along. No huge rush to land the diffHTML pr, I just feel bad leaving a broken build up.

@addyosmani
Member
addyosmani commented Apr 25, 2016 edited

This LGTM. Nice work, @samccone 🍰

@addyosmani addyosmani merged commit 48b34a0 into master Apr 25, 2016

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@addyosmani addyosmani deleted the sjs/update-test branch Apr 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment