fix: add --locked to uv export in Dockerfile - #1046
Conversation
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
Test Results69 tests 69 ✅ 1s ⏱️ Results for commit cc0bf8b. ♻️ This comment has been updated with latest results. |
WalkthroughChangesDocker build setup
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## dev #1046 +/- ##
=======================================
Coverage 96.07% 96.07%
=======================================
Files 1 1
Lines 331 331
=======================================
Hits 318 318
Misses 13 13 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
2c4f4f1 to
cc0bf8b
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@Dockerfile`:
- Line 13: Update the Dockerfile’s uv installation step to use an explicitly
approved, pinned uv version rather than the unversioned package; verify that the
selected version supports Python 3.14.5 and every target platform, and prefer a
pinned uv image tag or digest where applicable.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: 892a194b-c74d-4c52-8e7d-88e2d1f510ee
📒 Files selected for processing (1)
Dockerfile
Adds
--lockedtouv exportin the Dockerfile so it reads directly fromuv.lock(~0.95ms) instead of re-resolving dependencies on every build.This keeps the deterministic install from PR #1044 while avoiding the slow resolution step during container builds.
Summary by CodeRabbit