-
-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Add type hints for selenium.webdriver.remote.webdriver.WebDriver.__init__ #13775
Conversation
PR Description updated to latest commit (fb95163)
|
PR Review
✨ Review tool usage guide:Overview: The tool can be triggered automatically every time a new PR is opened, or can be invoked manually by commenting on any PR.
See the review usage page for a comprehensive guide on using this tool. |
PR Code Suggestions
✨ Improve tool usage guide:Overview:
See the improve usage page for a comprehensive guide on using this tool. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you, @adamtheturtle!
User description
Motivation and Context
This will help users of this class know earlier if they have made an error in creating a
WebDriver
object.Types of changes
Checklist
Type
enhancement
Description
WebDriver
class__init__
method to improve code readability and help with earlier error detection.Changes walkthrough
webdriver.py
Add Type Hints to WebDriver Initialization
py/selenium/webdriver/remote/webdriver.py
__init__
method parameters of theWebDriver
class.
command_executor
,keep_alive
,file_detector
, andoptions
now have explicit types.