Skip to content

[py] Fix default command_executor URL in RemoteWebDriver docstring - #17803

Merged
cgoldberg merged 2 commits into
SeleniumHQ:trunkfrom
vtvipul:fix-remote-webdriver-default-url
Jul 21, 2026
Merged

[py] Fix default command_executor URL in RemoteWebDriver docstring#17803
cgoldberg merged 2 commits into
SeleniumHQ:trunkfrom
vtvipul:fix-remote-webdriver-default-url

Conversation

@vtvipul

@vtvipul vtvipul commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

💥 What does this PR do?

The docstring stated the command_executor default as http://127.0.0.1:4444/wd/hub, but the actual default is http://127.0.0.1:4444 (no /wd/hub suffix). This PR corrects the docstring to match the code.

🔧 Implementation Notes

🤖 AI assistance

  • No substantial AI assistance used
  • AI assisted (complete below)
    • Tool(s):
    • What was generated:
    • I reviewed all AI output and can explain the change

🔄 Types of changes

  • Documentation

…cstring

The docstring stated the command_executor default as
'http://127.0.0.1:4444/wd/hub', but the actual default is
'http://127.0.0.1:4444' (no '/wd/hub' suffix). Correct the
docstring to match the code.
@CLAassistant

CLAassistant commented Jul 20, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@selenium-ci selenium-ci added the C-py Python Bindings label Jul 20, 2026
@qodo-code-review

Copy link
Copy Markdown
Contributor

PR Summary by Qodo

Fix RemoteWebDriver docstring default command_executor URL

📝 Documentation 🕐 Less than 5 minutes

Grey Divider

AI Description

• Correct RemoteWebDriver __init__ docstring default command_executor URL.
• Align documentation with the actual runtime default (no /wd/hub suffix).
High-Level Assessment

Updating the docstring to match the actual default is the correct and lowest-risk approach. Alternatives (changing the code default or adding compatibility logic) would be unnecessary because the intent is purely documentation alignment.

Files changed (1) +1 / -1

Documentation (1) +1 / -1
webdriver.pyFix command_executor default URL in RemoteWebDriver docstring +1/-1

Fix command_executor default URL in RemoteWebDriver docstring

• Updates the __init__ docstring to state the correct default command_executor value ('http://127.0.0.1:4444') rather than the outdated '/wd/hub' form. This keeps the documentation consistent with the actual default used by the implementation.

py/selenium/webdriver/remote/webdriver.py

@qodo-code-review

qodo-code-review Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Previous review results

Review updated until commit 60065e1 ⚖️ Balanced

Results up to commit 3af5533 ⚖️ Balanced


No changes from previous review

Qodo Logo

@cgoldberg cgoldberg changed the title [py] Fix incorrect default command_executor URL in RemoteWebDriver do… [py] Fix default command_executor URL in RemoteWebDriver docstring Jul 20, 2026
@qodo-code-review

Copy link
Copy Markdown
Contributor

Code review by qodo was updated up to the latest commit 60065e1

@cgoldberg cgoldberg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 LGTM, thanks!

@vtvipul Can you sign the CLA? #17803 (comment)

@vtvipul

vtvipul commented Jul 21, 2026

Copy link
Copy Markdown
Contributor Author

👍 LGTM, thanks!

@vtvipul Can you sign the CLA? #17803 (comment)

@cgoldberg Thanks for the review; I signed the CLA.

@cgoldberg
cgoldberg merged commit 8ad2bcb into SeleniumHQ:trunk Jul 21, 2026
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-py Python Bindings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants