Skip to content

Conversation

@cgoldberg
Copy link
Member

@cgoldberg cgoldberg commented Apr 8, 2025

User description

🔗 Related Issues

Fixes #15501

💥 What does this PR do?

This PR removes the logo (upper left on sidebar) from the Python API docs that are generated by Sphinx. This logo displays when I build the docs locally, but is missing/broken when the docs get published.

I'll eventually figure out the root cause and add it back, but it's easier to just remove the logo for now.

🔄 Types of changes

  • Bug fix (backwards compatible)

PR Type

Bug fix, Documentation


Description

  • Removed broken logo from Sphinx-generated Python API docs.

  • Commented out html_logo configuration in conf.py.

  • Prevents display of missing/broken logo in published docs.


Changes walkthrough 📝

Relevant files
Documentation
conf.py
Removed broken logo configuration in Sphinx `conf.py`       

py/docs/source/conf.py

  • Commented out the html_logo configuration.
  • Prevented the broken logo from appearing in the sidebar.
  • +1/-1     

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

    qodo-merge-pro bot commented Apr 8, 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 Apr 8, 2025
    @qodo-merge-pro
    Copy link
    Contributor

    qodo-merge-pro bot commented Apr 8, 2025

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    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.

    [🐛 Bug]: Broken logo image on Python API docs

    2 participants