chore(release): bump pixi.toml version in release.sh - #24
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
Included review availability: Your plan includes up to 1 review per rolling hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe release script now documents and performs top-level ChangesRelease version synchronization
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to This localized release-script change keeps the pixi.toml package version aligned during releases, with no actionable merge-blocking risk remaining after normal checks and review. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 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 |
The release script bumped CMakeLists.txt and CITATION.cff but left the [package] version in pixi.toml untouched, so it drifted from the tag (it sat at 0.2.0 through the 0.3.0 and 0.4.0 releases). Bump it in the same release commit, keeping the source dependency and the conda recipe in lockstep with the tag. The substitution is anchored at column 0 so it only matches the top-level version key, and is verified like the CMake bump. The block is guarded on the presence of that key so the same snippet is a clean no-op in sibling repos whose pixi.toml has no [package] section. Assisted-by: claude-code:claude-opus-4-8[1m]
444ef8f to
7abc2d3
Compare
|
@coderabbitai review |
✅ Action performedReview finished.
|
What
Teach
scripts/release.shto bump the[package]version inpixi.tomlalongsideCMakeLists.txtandCITATION.cff, and stage it into the release commit.Why
The script bumped
CMakeLists.txtandCITATION.cffbut never touchedpixi.toml, so its[package] versiondrifted from the git tag — it sat at0.2.0through both the0.3.0and0.4.0releases and had to be corrected by hand when cuttingv0.4.0. Thepixi.tomlcomment already asks to keep this version in lockstep with the conda recipe; this makes the release script do it automatically.Details
^version = "X.Y.Z"), so it only matches the top-level version key — not the inlineversion =fields of[package.build]/ host-dependency tables.Testing
bash -nandshellcheckpass clean.pixi.tomlmatches exactly one line (the[package]version).Summary by CodeRabbit
pixi.toml.