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

Another year, another change to how to deal with webdrivers #2410

Merged
merged 5 commits into from
Aug 12, 2023

Conversation

pablobm
Copy link
Collaborator

@pablobm pablobm commented Aug 4, 2023

Webdrivers breaking again (at least locally). Here's one of the fixes proposed at titusfortner/webdrivers#247

@pablobm
Copy link
Collaborator Author

pablobm commented Aug 4, 2023

Grr. Version 4.11 requires Ruby 3. Will have to try an alternative option.

$ gem install selenium-webdriver:4.11.0
Fetching selenium-webdriver-4.11.0.gem
ERROR:  Error installing selenium-webdriver:
        There are no versions of selenium-webdriver (= 4.11.0) compatible with your Ruby & RubyGems
        selenium-webdriver requires Ruby version >= 3.0. The current ruby version is 2.7.6.219.

Copy link
Contributor

@littleforest littleforest left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In other projects where I'm unable to upgrade selenium-webdriver past 4.9.0, I've been able to update webdrivers gem to 5.x and fix Webdrivers::Chromedriver.required_version = "114.0.5735.90"

@pablobm pablobm force-pushed the yet-another-webdrivers-fix branch from cda55d1 to c0bc158 Compare August 11, 2023 15:01
@pablobm
Copy link
Collaborator Author

pablobm commented Aug 11, 2023

Pushed again to try the option suggested by @littleforest

@pablobm pablobm force-pushed the yet-another-webdrivers-fix branch 3 times, most recently from 31ba12b to b9e0554 Compare August 11, 2023 15:37
@pablobm pablobm force-pushed the yet-another-webdrivers-fix branch from b9e0554 to a21fe5d Compare August 11, 2023 15:46
@pablobm
Copy link
Collaborator Author

pablobm commented Aug 11, 2023

OK! CircleCI didn't make this trivial, but it appears to work 🙂

Copy link
Contributor

@littleforest littleforest left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yay!

@pablobm pablobm merged commit 852b62c into thoughtbot:main Aug 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants