Skip to content

ci(release): stop release-please branch recursion#456

Merged
dc-tec merged 1 commit into
mainfrom
fix/release-please-main-loop
May 19, 2026
Merged

ci(release): stop release-please branch recursion#456
dc-tec merged 1 commit into
mainfrom
fix/release-please-main-loop

Conversation

@dc-tec

@dc-tec dc-tec commented May 19, 2026

Copy link
Copy Markdown
Owner

Summary

  • Excludes release-please--branches--* from the Release Please PR and Release Tag push triggers.
  • Adds a push-time guard that skips release-please when a push only changes release artifact files: CHANGELOG.md, .release-please-manifest.json, and charts/openbao-operator/Chart.yaml.
  • Keeps workflow_dispatch and empty Release-As: marker commits working because the guard only skips non-empty release-artifact-only pushes.

Related Issues

Follow-up to the recursive release-please PR loop that created #450-#455 from #395.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactor (code improvement/cleanup)
  • CI, release, or build tooling
  • Other maintenance

Risk and Compatibility

Release workflow only. This prevents recursive release-please runs on release-please head branches and avoids opening a new release PR from release-artifact-only merge commits. Manual Release-As dispatches and normal code/documentation changes still run release-please.

Verification

Reviewer Notes

The active main release PR #395 and its branch release-please--branches--main were intentionally left intact.

Checklist

  • My code follows the project style guide.
  • I have performed a self-review of my own code.
  • I have added or updated tests, or explained why tests are not needed.
  • I have updated documentation, or explained why docs are not needed.
  • I have updated generated artifacts, or confirmed none are affected.
  • I have checked that this change does not log or expose secrets, tokens, credentials, keys, or raw Secret data.
  • I have run the relevant local checks, or documented why they were not run.
  • Any dependent changes have been merged, published, or clearly called out.

Signed-off-by: Roel de Cort <roel.decort@adfinis.com>
@dc-tec dc-tec marked this pull request as ready for review May 19, 2026 05:55
@dc-tec dc-tec merged commit 6ab037a into main May 19, 2026
39 checks passed
@dc-tec dc-tec deleted the fix/release-please-main-loop branch May 19, 2026 05:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant