Skip to content

[py] Re-apply #15862 #15865

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

Merged
merged 2 commits into from
Jun 5, 2025
Merged

[py] Re-apply #15862 #15865

merged 2 commits into from
Jun 5, 2025

Conversation

cgoldberg
Copy link
Contributor

@cgoldberg cgoldberg commented Jun 5, 2025

User description

💥 What does this PR do?

re-merge #15862


PR Type

Bug fix, Documentation


Description

  • Update troubleshooting URLs in exception messages

  • Ensure links point to correct documentation anchors

  • Improve accuracy of error guidance for users


Changes walkthrough 📝

Relevant files
Documentation
exceptions.py
Correct troubleshooting links in exception error messages

py/selenium/common/exceptions.py

  • Updated anchor fragments in troubleshooting URLs for three exceptions
  • Ensured links match the correct documentation section IDs
  • Improved user guidance by pointing to accurate help resources
  • +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.
  • 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

    @cgoldberg cgoldberg merged commit 24a16fb into SeleniumHQ:trunk Jun 5, 2025
    4 of 5 checks passed
    @cgoldberg cgoldberg deleted the patch-2 branch June 5, 2025 15:20
    Copy link
    Contributor

    qodo-merge-pro bot commented Jun 5, 2025

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    @selenium-ci selenium-ci added the C-py Python Bindings label Jun 5, 2025
    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.

    3 participants