-
Notifications
You must be signed in to change notification settings - Fork 18
feat: upgrade to Python 3.13.3 #355
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
WalkthroughThe changes update the Python version from 3.12 to 3.13.3 across the GitHub Actions workflow, Dockerfile, and runtime specification files. Additional system packages are installed in the Docker build stage, and cleanup steps are enhanced to reduce the final image size. No application code or exported entities are modified. Changes
Possibly related PRs
Tip ⚡️ Faster reviews with caching
Enjoy the performance boost—your workflow just got faster. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
Coverage summary from CodacySee diff coverage on Codacy
Coverage variation details
Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: Diff coverage details
Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: See your quality gate settings Change summary preferences |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #355 +/- ##
=======================================
Coverage 89.18% 89.18%
=======================================
Files 2 2
Lines 111 111
=======================================
Hits 99 99
Misses 12 12
🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (1)
Dockerfile (1)
9-12: Consider pinning apt-get package versions for reproducibility
Static analysis flagged DL3008: installing packages without explicit versions can lead to non-deterministic builds. If strict reproducibility is a requirement, pin each package to a known version.🧰 Tools
🪛 GitHub Check: Codeac Code Quality
[warning] 9-9: DL3008
Pin versions in apt get install. Instead ofapt-get install <package>useapt-get install <package>=<version>
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
.github/workflows/python-app.yml(1 hunks)Dockerfile(1 hunks)runtime.txt(1 hunks)
🧰 Additional context used
🪛 GitHub Check: Codeac Code Quality
Dockerfile
[warning] 9-9: DL3008
Pin versions in apt get install. Instead of apt-get install <package> use apt-get install <package>=<version>
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: Codacy Static Code Analysis
🔇 Additional comments (6)
runtime.txt (1)
1-1: Consistent Python version update
The runtime specification has been correctly updated topython-3.13.3, aligning with the CI and Docker configurations..github/workflows/python-app.yml (2)
16-16: CI Python version bumped
UpdatingPYTHON_VERSIONto3.13.3ensures that lint, test, and container build jobs run against the new Python release.
15-17:Details
✅ Verification successful
Verify no lingering Python 3.12 references
To avoid inconsistency, search the repo for any remaining3.12mentions (e.g., in other workflows or docs):
🏁 Script executed:
#!/bin/bash # Exit non-zero if any "3.12" references are found if rg -l "3\.12" .; then echo "⚠️ Found leftover references to Python 3.12:" rg "3\.12" . exit 1 else echo "✅ No Python 3.12 references remain." fiLength of output: 98
✅ No Remaining Python 3.12 References
I ranrg -l "3\.12"across the repository and confirmed there are no leftover references to Python 3.12. All workflows and documentation now consistently use Python 3.13.3.Dockerfile (3)
3-3: Build stage base image upgraded
Switching the build stage topython:3.13.3-slim-bookwormaligns with the CI and runtime specs.
21-21: Runtime stage base image upgraded
The runtime image now usespython:3.13.3-slim-bookworm, keeping both stages in sync.
29-30: Wheelhouse-based installation and cleanup
Installing from prebuilt wheels and removing/app/wheelhouseis a solid pattern to minimize image size and avoid network access at runtime.



This change is
Summary by CodeRabbit