Skip to content

Fix Python interpreter selection for reproducible installs#7

Open
shahabajsheikh-ops wants to merge 1 commit into
devrev:mainfrom
shahabajsheikh-ops:shah/python-compatibility
Open

Fix Python interpreter selection for reproducible installs#7
shahabajsheikh-ops wants to merge 1 commit into
devrev:mainfrom
shahabajsheikh-ops:shah/python-compatibility

Conversation

@shahabajsheikh-ops

@shahabajsheikh-ops shahabajsheikh-ops commented Jul 25, 2026

Copy link
Copy Markdown

Summary

  • prevent uv sync from selecting unsupported Python 3.14
  • declare and test the supported Python 3.12-3.13 range
  • align setup documentation and dataset manifest digests

Reproduction

On macOS, make install selected CPython 3.14.5 and failed while building litellm==1.92.0 because its locked PyO3 dependency supports at most Python 3.13. With this change, Python 3.14 receives a clear incompatible-version message while Python 3.12 and 3.13 install successfully.

Related issue

https://app.devrev.ai/devrev/works/ISS-350288

Validation

  • uv sync --python 3.12 --extra dev --frozen
  • make validate on Python 3.12.13
  • uv sync --python 3.13 --extra dev --frozen
  • make validate on Python 3.13.5
  • Python 3.14.5 negative compatibility check
  • uv lock --check
  • git diff --check

Author: Shahabaj Sheikh

@shahabajsheikh-ops

shahabajsheikh-ops commented Jul 25, 2026

Copy link
Copy Markdown
Author

The repository CI passes on both Python 3.12 and 3.13.

Update:

  • DevRev work item ISS-350288 was created for this defect and added to the PR description.
  • validate_description now passes.
  • patching_sla_check still stops while loading repository AWS credentials, before checking the patch. This requires a trusted repository run or maintainer override.

— Shahabaj Sheikh

@shahabajsheikh-ops

shahabajsheikh-ops commented Jul 25, 2026

Copy link
Copy Markdown
Author

@mira-mohammad @jonmaaku-rev @roycabezas @nimit2801 could you please review this contribution?

The Python 3.12 and 3.13 CI jobs, global checks, and description validation all pass. The only remaining required check, patching_sla_check, stops while configuring repository AWS credentials before inspecting the patch because this pull request originates from a fork. Could you please rerun or waive that check from a trusted repository context?

The corresponding DevRev work item is ISS-350288.

— Shahabaj Sheikh

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.

1 participant