-
-
Notifications
You must be signed in to change notification settings - Fork 6
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
Documentation Bug: platformName
must be lowecase windows
#284
Comments
This is due to a recent breaking change in Chromedriver itself, which this project is merely a thin interface for. Any help in fixing casing issues in docs/examples via PRs would be very much appreciated! |
(whilst I understand the issue isn't in this project, I am mentioning here as it is the "interface" into Chromedriver for many) |
Thank you! Would you like to submit a PR with these updates to the documentation? |
Sure can. Can I push a branch to this repo ? Do you know where the actual docs are for the Chromedriver releases ? |
The process for opening a PR on an open source repo is to first fork the repo, push to your fork, and then create a PR. you can also simply navigate to the file where you want to make a change, and propose a PR from the github UI itself (which is easiest for docs changes like this). I'm not sure where you'd want to update the docs exactly, presumably wherever you saw the (now-)erroneous examples that led you to create this issue? |
All documentation and error messages for
platformName
mention the windows platform in Titlecase:Windows
. But the driver only accepts windows as lowercase:windows
.This took me a day to figure out.
These are the steps to reproduce from my Stackoverflow question:
The text was updated successfully, but these errors were encountered: