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

Replace webdrivers with selenium-webdriver #60

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jcoyne
Copy link
Contributor

@jcoyne jcoyne commented Oct 13, 2023

No description provided.

@marlo-longley
Copy link
Contributor

marlo-longley commented Sep 23, 2024

@jcoyne I tried to review this locally but ran into errors I think having to do with the version of selenium-webdriver

2024-09-23 14:53:58 WARN Selenium [:selenium_manager] Error getting version of chromedriver 128. Retrying with chromedriver 127 (attempt 1/5) 
2024-09-23 14:53:58 WARN Selenium [:selenium_manager] Error getting version of chromedriver 127. Retrying with chromedriver 126 (attempt 2/5) 
2024-09-23 14:53:58 WARN Selenium [:selenium_manager] Error getting version of chromedriver 126. Retrying with chromedriver 125 (attempt 3/5) 
2024-09-23 14:53:58 WARN Selenium [:selenium_manager] Error getting version of chromedriver 125. Retrying with chromedriver 124 (attempt 4/5) 
2024-09-23 14:53:58 WARN Selenium [:selenium_manager] Error getting version of chromedriver 124. Retrying with chromedriver 123 (attempt 5/5) 
2024-09-23 14:53:58 ERROR Selenium [:selenium_manager] The chromedriver version cannot be discovered 

Based on our other apps I tried it out here (a PR where I am doing other dependency updates) with 4.11 which seemed to fix it: 8af3ae4

Let me know if that seems OK.

@jcoyne
Copy link
Contributor Author

jcoyne commented Sep 23, 2024

@marlo-longley which version did you have installed? ~> 4.11 should be the same resolution (4.25.0) as the unscoped requirement.

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