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

Fixing #592 win32 downloaded on 64-bit windows #635

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

bandophahita
Copy link

No description provided.

@pep8speaks
Copy link

pep8speaks commented Oct 9, 2023

Hello @bandophahita! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 125:80: E501 line too long (86 > 79 characters)

Comment last updated at 2023-10-13 16:52:32 UTC

@vhalis
Copy link

vhalis commented Nov 30, 2023

I just encountered this as well and submitted #647 before checking what else had been done :) ... one thing to consider is that versions before 115 can be downloaded from the default URL https://chromedriver.storage.googleapis.com and they don't have win64 versions.

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.

None yet

3 participants