-
Notifications
You must be signed in to change notification settings - Fork 101
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Install the latest version of Chrome on Travis. #117
Comments
is there a way to get travis to run multiple browsers? or is that only meaningfull if we start distributing a polyfill package #108 |
Well this doesn't look good! - https://twitter.com/domenic/status/502571560944021506 |
Yeah and another issue is that even if I do figure it out, it will add a substantial amount of time to the build. |
Installing a specific version of Firefox is much easier, it seems like: https://docs.travis-ci.com/user/ci-environment/#Firefox |
Finally found the issue of why Chromium browser is stuck on version 37: travis-ci/travis-ci#3475 |
I have it working, seems a little crazy for the |
Takes the build up to 3+ minutes. |
There might be some steps we can take out too. I just copied from some place that seemed to have it worked out. |
I kind of did hit and trial and got Chrome 47 working. :) And it passes with all the tests for |
Let me know if what I did is correct solves the problem. If you wish I can send the PR for it. :) |
So the only question I have from https://github.com/praveenpuglia/bliss/blob/th-test/.travis.yml is why do we need Other than that, I say yeah, submit a PR for it. I'll delete my branch. Good job. |
oh you don't. I did it just to test the commit with my own travis build. let me send a PR. Thanks! :) Just to be confirm, I am sending PR only for the travis update right? or merge the changes as well ? |
Sorry, I meant |
Okay. When I did normal installation, it failed with error indicating those packages were chrome dependencies and could not be found. |
Ok. Just making sure. |
…um 37.0.2062) Thanks to LeaVerou/bliss#117
Hope it helps:
|
…um 37.0.2062) Thanks to LeaVerou/bliss#117
…um 37.0.2062) Thanks to LeaVerou/bliss#117
…um 37.0.2062) Thanks to LeaVerou/bliss#117
…um 37.0.2062) Thanks to LeaVerou/bliss#117
…um 37.0.2062) Thanks to LeaVerou/bliss#117
…um 37.0.2062) Thanks to LeaVerou/bliss#117
…um 37.0.2062) Thanks to LeaVerou/bliss#117
…um 37.0.2062) Thanks to LeaVerou/bliss#117
…um 37.0.2062) Thanks to LeaVerou/bliss#117
Seems related to #109. I will fork a branch and see what I can do to get this working.
The text was updated successfully, but these errors were encountered: