Skip to content

chore: update python version r1 - #29

Merged
Sagargupta16 merged 1 commit into
mainfrom
chore/maint-r1-4
Mar 5, 2026
Merged

chore: update python version r1#29
Sagargupta16 merged 1 commit into
mainfrom
chore/maint-r1-4

Conversation

@Sagargupta16

Copy link
Copy Markdown
Owner

Maintenance update

Copilot AI review requested due to automatic review settings March 5, 2026 02:41
@Sagargupta16
Sagargupta16 merged commit c2d20c0 into main Mar 5, 2026
@Sagargupta16
Sagargupta16 deleted the chore/maint-r1-4 branch March 5, 2026 02:41
@sonarqubecloud

sonarqubecloud Bot commented Mar 5, 2026

Copy link
Copy Markdown

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR introduces a new .python-version file at the repository root, intended to pin the local Python interpreter version for tools like pyenv and uv.

Changes:

  • Adds a .python-version file pinning Python to version 3.11

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .python-version
@@ -0,0 +1 @@
3.11

Copilot AI Mar 5, 2026

Copy link

Choose a reason for hiding this comment

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

The .python-version file sets the Python version to 3.11, but the rest of the project consistently uses Python 3.12:

  • CI workflow (.github/workflows/ci.yml, line 22) pins python-version: '3.12'
  • render.yaml (line 11) sets PYTHON_VERSION: "3.12.0" for the production deployment
  • pyproject.toml (line 72) configures mypy with python_version = "3.12"

This mismatch means local environments using pyenv or uv (which both respect .python-version) will run Python 3.11, while CI and production run Python 3.12. This could lead to hard-to-debug discrepancies (e.g., type-checking inconsistencies, syntax or behaviour differences). The value should be updated to 3.12 to be consistent with the rest of the stack.

Suggested change
3.11
3.12

Copilot uses AI. Check for mistakes.
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