chore: auto-merge safe GitHub Actions updates with a release-age cooldown - #89
Conversation
…down Reduce Renovate "review rot" for GitHub Actions while preserving safety. - Auto-merge github-actions minor/patch/digest/pinDigest updates (platformAutomerge). Majors are excluded, so the existing major rule's automerge:false keeps breaking upgrades human-reviewed. - Add an org-wide minimumReleaseAge cooldown (5 days) with internalChecksFilter:strict so we never auto-adopt a freshly-published (possibly compromised) release; genuine security fixes are exempt via vulnerabilityAlerts.minimumReleaseAge:0. CI/status checks still gate every merge. For auto-merge to actually fire, Renovate must also be added as a bypass actor on the org "Baseline" ruleset (separate org-settings step); until then these PRs are marked auto-mergeable but still wait for the required review. Co-Authored-By: Claude <noreply@anthropic.com> Ref: EC-2185
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review. 📝 WalkthroughWalkthroughRenovate now delays standard releases for five days, bypasses that delay for vulnerability alerts, and automatically merges non-major GitHub Actions updates. Major GitHub Actions updates remain subject to manual review. ChangesRenovate policy updates
Priority: ⬇️ Low — Defer this change because it is limited to shared Renovate automation for eligible GitHub Actions updates, with vulnerability alerts already exempt from the cooldown. Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to The Renovate policy adds a five-day standard release cooldown, exempts vulnerability alerts, and auto-merges eligible non-major GitHub Actions updates while retaining manual review for major upgrades. No current merge-blocking risk is identified. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Warning Some tools did not complete. Review the errors below. 🔧 Biome (2.5.8)config/renovate/renovate.jsonFile contains syntax errors that prevent linting: Line 4: Expected a property but instead found '// Base configuration presets - these provide sensible defaults'.; Line 5: End of file expected; Line 5: End of file expected; Line 6: Expected an array, an object, or a literal but instead found '// Run renovate each day before 4AM'.; Line 7: expected ... [truncated 5253 characters] ... ile expected; Line 81: Expected a property but instead found '// Tekton: Safe to auto-merge (infrastructure as code)'.; Line 80: End of file expected; Line 81: End of file expected; Line 82: End of file expected; Line 82: End of file expected; Line 82: End of file expected; Line 82: End of file expected; Line 83: End of file expected; Line 83: End of file expected; Line 83: End of file expected; Line 83: End of file expected; Line 84: End of file expected; Line 84: End of file expected; Line 84: End of file expected; Line 84: End of file expected; Line 85: End of file expected; Line 85: End of file expected; Line 85: End of file expected; Line 85: End of file expected; Line 86: End of file expected; Line 86: End of file expected; Line 86: End of file expected; Line 86: End of file expected Comment |
PR Summary by QodoAuto-merge safe GitHub Actions updates after a release cooldown
AI Description
Diagram
High-Level Assessment
Files changed (1)
|
Code Review by Qodo🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)
Great, no issues found!Qodo reviewed your code and found no material issues that require reviewTip of the day💡 Did you know, you can tweak Display preferences with a live preview to see your comment before it ships |
What
Two changes to the shared org Renovate preset (
config/renovate/renovate.json):github-actionsminor/patch/digest/pinDigestupdates (automerge: true+platformAutomerge: true). Majors are excluded, so the existing global major rule (automerge: false, 🚨 label) keeps breaking upgrades human-reviewed.minimumReleaseAge: "5 days"+internalChecksFilter: "strict", withvulnerabilityAlerts.minimumReleaseAge: "0 days"so genuine security fixes skip the wait.Why
Now that conforma is 100% SHA-pinned, Renovate opens a continuous stream of digest/minor/patch GitHub-Actions PRs to keep pins current. Each currently needs a human approval, so they pile up ("review rot") and pins risk going stale. This lets the safe ones flow automatically — but never a fresh release (cooldown guards against a compromised version being auto-adopted before the community can detect/yank it).
Safety
automerge: false).Note — separate follow-up required for auto-merge to actually fire
Renovate’s auto-merge waits for the required review; it does not bypass it. So this PR marks the safe PRs auto-mergeable, but they will only merge once Renovate is added as a bypass actor on the org “Baseline” ruleset (scoped to pull requests) — an org-settings change tracked in EC-2185 alongside this PR.
Ref: EC-2185
Co-Authored-By: Claude noreply@anthropic.com