Skip to content

Conversation

bernhardkaindl
Copy link
Contributor

@bernhardkaindl bernhardkaindl commented Aug 26, 2025

Brief summary:

  • Fix running pytype analysis: Needs Python 3.11 (3.12 is not supported yet)

Pull Request Overview

This PR addresses pytype analysis by enforcing the Python 3.11 requirement, as pytype lacks support for Python 3.12 and newer.

The changes update the CI configuration and add runtime validation to ensure a proper execution environment.

  • Moves pytype execution from Python 3.12 to Python 3.11 environment in tox configuration
  • Adds runtime validation to ensure pytype runs only with the required Python version
  • Improves error reporting when pytype fails without producing parseable results

Reviewed Changes

Copilot reviewed both changed files in this pull request and iteratively generated several comments in the separate PR for Copilot Code review: bernhard-xapi#10.

All further Copilot comments are now fixed, and it has added no additional comments so far.

File Description
tox.ini Moves pytype from py312 to py311 environment to ensure compatibility
pytype_runner.py Adds Python version validation and improves error handling for pytype failures

@bernhardkaindl bernhardkaindl force-pushed the fix-running-pytype-analysis branch from 2f1a479 to 4a6dd96 Compare August 26, 2025 15:19

This comment was marked as off-topic.

@coveralls

This comment was marked as off-topic.

Copilot

This comment was marked as duplicate.

Signed-off-by: Bernhard Kaindl <bernhard.kaindl@cloud.com>
@bernhardkaindl bernhardkaindl force-pushed the fix-running-pytype-analysis branch from a5edf74 to d2a1e2e Compare August 27, 2025 14:33
@bernhardkaindl bernhardkaindl merged commit 9969603 into xenserver:master Aug 28, 2025
10 checks passed
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.

3 participants