Skip to content

Conversation

@adolfoarmas
Copy link
Contributor

@adolfoarmas adolfoarmas commented Jun 5, 2025

User description

Update urls to the correct troubleshooting links:

-https://www.selenium.dev/documentation/webdriver/troubleshooting/errors/#nosuchelementexception

-https://www.selenium.dev/documentation/webdriver/troubleshooting/errors/#staleelementreferenceexception

-https://www.selenium.dev/documentation/webdriver/troubleshooting/errors/#invalidselectorexception

🔄 Types of changes

  • Cleanup (formatting, renaming)

PR Type

Documentation


Description

  • Update error documentation URLs in exception messages

  • Ensure links point to correct troubleshooting sections

  • Minor cleanup for consistency in exception classes


Changes walkthrough 📝

Relevant files
Documentation
exceptions.py
Correct error documentation URLs in exception classes       

py/selenium/common/exceptions.py

  • Updated URLs in exception constructors to correct troubleshooting
    anchors
  • Ensured consistency in error documentation links for three exceptions
  • No logic or functional changes, only documentation references updated
  • +3/-3     

    Need help?
  • Type /help how to ... in the comments thread for any questions about Qodo Merge usage.
  • Check out the documentation for more information.
  • @CLAassistant
    Copy link

    CLAassistant commented Jun 5, 2025

    CLA assistant check
    All committers have signed the CLA.

    @qodo-merge-pro
    Copy link
    Contributor

    qodo-merge-pro bot commented Jun 5, 2025

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ No major issues detected

    @selenium-ci selenium-ci added the C-py Python Bindings label Jun 5, 2025
    @qodo-merge-pro
    Copy link
    Contributor

    qodo-merge-pro bot commented Jun 5, 2025

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    @cgoldberg cgoldberg self-requested a review June 5, 2025 15:04
    Copy link
    Member

    @cgoldberg cgoldberg left a comment

    Choose a reason for hiding this comment

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

    LGTM 👍
    Thanks for the contribution!

    Note: CI jobs are failing on this PR, but I don't see how it could be related to these changes. I will investigate separately.

    @cgoldberg cgoldberg merged commit 2d2cb13 into SeleniumHQ:trunk Jun 5, 2025
    14 of 19 checks passed
    Copy link
    Member

    @Delta456 Delta456 left a comment

    Choose a reason for hiding this comment

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

    Thanks for updating the links.

    You also need to make a change here so that the CI doesn't fail.

    @cgoldberg
    Copy link
    Member

    @Delta456 ouch.. I already merged it. I'll fix that now. 1 sec.

    @Delta456
    Copy link
    Member

    Delta456 commented Jun 5, 2025

    @Delta456 ouch.. I already merged it. I'll fix that now. 1 sec.

    Please don't revert it as I made a PR at #15863

    @adolfoarmas
    Copy link
    Contributor Author

    @Delta456

    You also need to make a change here so that the CI doesn't fail.

    Good to know, thank you!

    @cgoldberg cgoldberg mentioned this pull request Jun 5, 2025
    cgoldberg added a commit that referenced this pull request Jun 5, 2025
    @cgoldberg
    Copy link
    Member

    @Delta456 it should be all set now. thanks

    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Labels

    Projects

    None yet

    Development

    Successfully merging this pull request may close these issues.

    5 participants