chore: support uv.lock and version.txt in release config#274
chore: support uv.lock and version.txt in release config#274SoulPancake merged 1 commit intomainfrom
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #274 +/- ##
=======================================
Coverage 69.85% 69.85%
=======================================
Files 140 140
Lines 10748 10748
=======================================
Hits 7508 7508
Misses 3240 3240 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
Updates the repository’s release-please configuration to keep additional version sources (a plain version file and uv’s lockfile) in sync during releases.
Changes:
- Add
VERSION.txtanduv.locktorelease-please-config.jsonextra-files. - Add the
x-release-please-versionmarker toVERSION.txtso release-please can reliably update it.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| release-please-config.json | Adds new extra-files entries so release-please updates VERSION.txt and uv.lock during release PRs. |
| VERSION.txt | Adds the release-please version marker on the version line. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
WalkthroughVersion management configuration is updated by appending a release-please directive to VERSION.txt and extending the release-please configuration to track version updates across additional files including VERSION.txt and the uv.lock package lockfile. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Suggested reviewers
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Warning Review ran into problems🔥 ProblemsTimed out fetching pipeline failures after 30000ms 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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@release-please-config.json`:
- Line 32: The JSONPath for the uv.lock entry is wrong: change the filter in the
release-please-config.json entry where path "uv.lock" and jsonpath
"$.package[?(@.name.value=='openfga-sdk')].version" is used so it checks the
string field `name` rather than `name.value`; update the JSONPath to use @.name
(i.e., the filter should compare @.name to 'openfga-sdk') so the package version
for openfga-sdk is correctly matched and synced.
🪄 Autofix (Beta)
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: CHILL
Plan: Pro
Run ID: 2ee7e1e1-aecb-48b0-aea0-a66b957ed5a6
📒 Files selected for processing (2)
VERSION.txtrelease-please-config.json
Description
What problem is being solved?
How is it being solved?
What changes are made to solve it?
References
Review Checklist
mainSummary by CodeRabbit