Skip to content

Conversation

@bernhardkaindl
Copy link

No description provided.

This comment was marked as outdated.

@bernhardkaindl bernhardkaindl force-pushed the docs-update-badges-add-blueprint-for-render branch from 36a9f0a to fc4447c Compare August 29, 2025 14:29
@bernhard-xapi bernhard-xapi deleted a comment from Copilot AI Aug 29, 2025
@bernhard-xapi bernhard-xapi deleted a comment from Copilot AI Aug 29, 2025
@bernhard-xapi bernhard-xapi deleted a comment from Copilot AI Aug 29, 2025
@bernhardkaindl bernhardkaindl force-pushed the docs-update-badges-add-blueprint-for-render branch from fc4447c to c237dbc Compare August 29, 2025 14:30
@bernhardkaindl bernhardkaindl requested a review from Copilot August 29, 2025 14:32
@bernhard-xapi bernhard-xapi deleted a comment from coveralls Aug 29, 2025

This comment was marked as outdated.

@bernhardkaindl bernhardkaindl force-pushed the docs-update-badges-add-blueprint-for-render branch from c237dbc to dff763a Compare August 29, 2025 14:37
@bernhard-xapi bernhard-xapi deleted a comment from codecov bot Aug 29, 2025
@bernhard-xapi bernhard-xapi deleted a comment from Copilot AI Aug 29, 2025
@bernhard-xapi bernhard-xapi deleted a comment from Copilot AI Aug 29, 2025
@bernhard-xapi bernhard-xapi deleted a comment from Copilot AI Aug 29, 2025
@bernhardkaindl bernhardkaindl force-pushed the docs-update-badges-add-blueprint-for-render branch from dff763a to 72a7bd0 Compare August 29, 2025 14:46
@bernhardkaindl bernhardkaindl requested a review from Copilot August 29, 2025 14:46

This comment was marked as outdated.

@bernhardkaindl bernhardkaindl force-pushed the docs-update-badges-add-blueprint-for-render branch from 72a7bd0 to 9f543a6 Compare August 29, 2025 14:48
@bernhard-xapi bernhard-xapi deleted a comment from Copilot AI Aug 29, 2025
@bernhardkaindl bernhardkaindl requested a review from Copilot August 29, 2025 14:49
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request adds support for deploying documentation to Render.com and updates project metadata to reflect current Python version support. The changes modernize the project's documentation hosting and simplify Python version requirements.

  • Adds Render.com deployment configuration for automatic documentation hosting
  • Updates Python version requirements to support only 3.6, 3.11, 3.12, and 3.13
  • Adds documentation badges and updates project branding to "XenServer Python libs for Dom0"

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
render.yaml New Render.com service configuration for static documentation hosting
pyproject.toml Updated Python version requirements and classifiers
docs/source/index.rst Added badges and updated documentation title
docs/source/conf.py Added MyST heading anchors configuration
docs/requirements.txt Added six dependency for documentation build
README.md Updated badges, title, and project description
.github/workflows/main.yml Added webhook trigger for Render.com deployment

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment on lines +101 to +103
if: |
env.deploy_webhook_url && github.ref == env.docs_deploy_branch &&
matrix.python-version == env.PYTHON_VERSION_USED_FOR_COVERAGE
Copy link

Copilot AI Aug 29, 2025

Choose a reason for hiding this comment

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

The condition references env.PYTHON_VERSION_USED_FOR_COVERAGE but this environment variable is not defined in this workflow. This will cause the condition to always evaluate to false, preventing the deployment webhook from triggering.

Copilot uses AI. Check for mistakes.
@bernhard-xapi bernhard-xapi deleted a comment from Copilot AI Aug 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants