Skip to content

*: improve automated workflows - #31

Open
KaloyanTanev wants to merge 5 commits into
obolfrom
kalo/improve-automated-workflows
Open

*: improve automated workflows#31
KaloyanTanev wants to merge 5 commits into
obolfrom
kalo/improve-automated-workflows

Conversation

@KaloyanTanev

Copy link
Copy Markdown

Some small QoL improvements over the workflows.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Improves the reliability and observability of the automated “rebase + tag” workflows by switching from output-grep heuristics to exit-code based success detection, adding a post-rebase verification step before tagging, and tightening what gets pushed.

Changes:

  • Capture git rebase output once, use the exit code to set a success output, and abort failed rebases to leave a clean working tree for retries.
  • Add a git range-diff verification gate to ensure *-obol tags are patch-identical to “upstream tag + obol changes” before tagging.
  • Push only the newly created tag ref instead of all tags.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
.github/workflows/tag.yml Adds exit-code based rebase handling, introduces range-diff verification gating for tagging, and pushes only the new tag ref.
.github/workflows/sync-rebase.yml Switches rebase success detection to exit code, persists output cleanly, and gates push on a boolean success output.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/tag.yml
Comment thread .github/workflows/tag.yml
@KaloyanTanev
KaloyanTanev force-pushed the kalo/improve-automated-workflows branch from 57c79e7 to fa85f3c Compare July 14, 2026 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants