Skip to content

fix: consolidate release workflow and publish draft releases#7

Merged
b3nk3 merged 1 commit intomainfrom
feature/conductor-workspace
Jan 24, 2026
Merged

fix: consolidate release workflow and publish draft releases#7
b3nk3 merged 1 commit intomainfrom
feature/conductor-workspace

Conversation

@b3nk3
Copy link
Copy Markdown
Owner

@b3nk3 b3nk3 commented Jan 19, 2026

Fixes the Homebrew 404 issue by ensuring releases are published before assets are accessible. Merged tag.yaml into release.yaml for unified tag creation and release workflow with draft publication.

Summary by CodeRabbit

  • Chores
    • Streamlined release process by integrating version bumping and tag creation directly into the release workflow, consolidating previously separate automation steps.

✏️ Tip: You can customize this high-level summary in your review settings.

Merged tag.yaml into release.yaml to create a single unified workflow
that handles tag creation, GoReleaser execution, and draft publication.
This fixes the Homebrew 404 issue where draft releases were not publicly
accessible.

Changes:
- Consolidated tag creation and release into single workflow
- Added git fetch to pull newly created tags
- Pass GORELEASER_CURRENT_TAG explicitly to GoReleaser
- Publish draft releases after asset upload completes
- Deleted separate tag.yaml workflow

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Jan 19, 2026

📝 Walkthrough

Walkthrough

The release and tagging workflows are consolidated into a single release workflow. Version bumping and tag creation functionality, previously in a separate workflow, is now integrated as a conditional first step. Subsequent build and publish steps execute only if a new tag is successfully created.

Changes

Cohort / File(s) Change Summary
GitHub Actions Workflow Consolidation
\.github/workflows/release.yaml`, \.github/workflows/tag.yaml``
Consolidated tagging and release workflows. Release workflow now includes a new conditional "Bump version and create tag" step using mathieudutour/github-tag-action@v6.2. Subsequent steps (Go setup, GoReleaser, publish) now gate on steps.tag.outputs.new_tag. Tag workflow removed; its functionality merged into release workflow. Renamed workflow from "goreleaser" to "Release" and job from "goreleaser" to "release". Added GORELEASER_CURRENT_TAG environment variable and "Publish release" step using gh release edit.

Sequence Diagram

sequenceDiagram
    participant GHA as GitHub Actions
    participant TagAction as Tag Action<br/>(mathieudutour)
    participant GoReleaser as GoReleaser
    participant GHCli as GitHub CLI<br/>(gh release edit)

    GHA->>TagAction: Trigger tag creation<br/>(on: push to main)
    TagAction->>TagAction: Bump version & create tag
    TagAction-->>GHA: Output new_tag
    
    alt new_tag exists
        GHA->>GoReleaser: Set up Go & fetch tag
        GHA->>GoReleaser: Run GoReleaser<br/>(with GORELEASER_CURRENT_TAG)
        GoReleaser-->>GHA: Release artifacts
        GHA->>GHCli: Publish release
    else no new tag
        GHA->>GHA: Skip build & publish
    end
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 Two workflows became one, oh what grace!
Tagging and releasing share a space.
Conditional gates guard the way,
Making deployments cleaner each day! 🏷️✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title accurately describes the main changes: consolidating the release workflow (merging tag.yaml into release.yaml) and publishing draft releases to fix the Homebrew 404 issue.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@b3nk3 b3nk3 merged commit bc80b5d into main Jan 24, 2026
5 checks passed
@b3nk3 b3nk3 deleted the feature/conductor-workspace branch January 24, 2026 17:43
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.

1 participant