close
Skip to content

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

Merged
pablobm merged 5 commits into
thoughtbot:mainfrom
pablobm:yet-another-webdrivers-fix
Aug 12, 2023
Merged

Another year, another change to how to deal with webdrivers#2410
pablobm merged 5 commits into
thoughtbot:mainfrom
pablobm:yet-another-webdrivers-fix

Conversation

@pablobm
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Collaborator Author

pablobm commented Aug 11, 2023

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

Copy link
Copy Markdown
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