Skip to content
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

Specify GitHub repository instead of the path #2267

Open
wants to merge 1 commit into
base: master
from

Conversation

@y-yagi
Copy link
Contributor

y-yagi commented Oct 31, 2019

I fixed this because I want to work a test against ruby-head.
https://travis-ci.org/teamcapybara/capybara/jobs/605383442

But if the purpose of this Gemfile is to check the test locally, feel free to close this. Thanks.

@twalpole

This comment has been minimized.

Copy link
Member

twalpole commented Oct 31, 2019

The purpose of this was to use the edge version of selenium-webdriver during the travis tests. Selenium-webdriver requires (or used to require) to be built in order for the JS atoms to be available, and couldn't be used directly from the repo. I did some work on changing that in the selenium repo a while back but haven't verified whether they've been kept up to date in a couple of months (I will have some time tomorrow/this weekend). Since it required to be built, the package was downloaded on travis, then built, and then used locally.

@y-yagi

This comment has been minimized.

Copy link
Contributor Author

y-yagi commented Nov 1, 2019

Thanks for your explanation! I misunderstood the purpose of that build.
I will close this. By the way, how do you feel add build that ruby-head with default Gemfile?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.