Repository navigation
Conversation
Follow-up to #37, implementing the rules agreed in #35: - patch updates for dependencies still below 1.0.0 - all updates of ghcr.io/epics-containers/ioc-* images, which use CalVer so a year rollover is a "major" bump Refs #35 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014GsvY8MPA1WdKubZ76YVcd
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. Warning Review limit reachedNext included review available in 17 minutes. View limit detailsLimit details: You’ve used the included review currently available. This review ran on the open-source allowance, not this organization's plan, because the pull request author doesn't have an assigned seat. Waiting won't change this — ask an organization admin to assign them a seat, or add seats in Billing if every seat is already assigned, then retry. Review configuration: ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe Renovate configuration adds two automerge rules: one for patch updates to packages with current versions beginning with ChangesRenovate automerge rules
Estimated code review effort: 2 (Simple) | ~8 minutes Merge Risk: ⚪ Minimal · up to The new rules match the intended dependency updates. No concrete merge-blocking behavior change is evident. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
…teways chart - Add matchCurrentVersion /^20\d\d\./ to the ioc-* image rule so it only automerges majors for CalVer-pinned images (e.g. not ioc-asyn-runtime:4.46). - Add a matching rule for the epics-gateways helm chart, which is also CalVer and would otherwise need a manual merge on a year rollover. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Jd9gGzEQubx2j1kD6Q2s8a
|
Independent review by a Claude Fable agent
Checked with Review and replies by Claude Code. |
|
Keeping this open for now. I'd like to prove automerge on p47 first, with an overnight test as the safety net: epics-containers/p47-services#202 (comment) |
Fixes #35
#37 added automerge for minor and patch updates with current version >= 1.0.0. The follow-ups discussed in #35 (reopened 2025-05-22) were never done:
ghcr.io/epics-containers/ioc-*images. Generic IOCs use CalVer, so a year rollover counts as "major". The thread concluded "stick to datever for IOCs and always deploy a new tag".blueapi no longer needs special handling because it is at 1.x now and its updates already automerge (#205, #206).
Automerge still waits for the
verify_ioc_instancescheck. For example, #209 (ioc-pmac-runtime 2026.8.1 → 2026.9.1) is failing verify and would not merge.Testing
renovate-config-validator renovate.json: config validated successfully.If you'd rather put these rules in services-template-helm's
template/renovate.jsonfor every beamline, tell me and I'll move them there.🤖 Generated with Claude Code
https://claude.ai/code/session_014GsvY8MPA1WdKubZ76YVcd
Summary by CodeRabbit