Skip to content

Fetch tags before GoReleaser and skip releases for workflow-only pushes - #42

Merged
aurimasbutkus merged 1 commit into
mainfrom
build-release-fixes
Jun 12, 2026
Merged

Fetch tags before GoReleaser and skip releases for workflow-only pushes#42
aurimasbutkus merged 1 commit into
mainfrom
build-release-fixes

Conversation

@aurimasbutkus

@aurimasbutkusaurimasbutkus commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores
    • Optimized release workflow to prevent unnecessary builds when workflow files are modified.
    • Enhanced tag handling during the release process.

@aurimasbutkus
aurimasbutkus requested a review from a team as a code ownerJune 12, 2026 14:35
@coderabbitai

coderabbitaiBot commented Jun 12, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 9f2027c0-dad5-4312-b3b1-80d85c8863e6

📥 Commits

Reviewing files that changed from the base of the PR and between 33e04b9 and 544cba9.

📒 Files selected for processing (1)
  • .github/workflows/build-release.yaml

📝 Walkthrough

Walkthrough

This pull request optimizes the release workflow by preventing redundant runs when only GitHub workflow files change, and by ensuring Git tags are properly fetched after the version bump step completes.

Changes

Release Workflow Improvements

Layer / File(s)Summary
Workflow trigger optimization and tag fetching
.github/workflows/build-release.yaml
Workflow push trigger on main now ignores .github/** path changes via paths-ignore to prevent unnecessary runs on configuration edits. A conditional git fetch --force --tags step is added after the tag bump action when steps.bump.outputs.new_tag is non-empty, ensuring tags are available for subsequent release tooling.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • hostinger/api-cli#36: Both PRs modify .github/workflows/build-release.yaml around the release/tag-bump flow; this PR adds git tag fetching after bump, while the other PR changes checkout and tag-bump handling.

Suggested reviewers

  • zygintas
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe title accurately describes the two main changes: fetching tags before GoReleaser and skipping releases for workflow-only pushes via paths-ignore.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch build-release-fixes

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

@aurimasbutkus
aurimasbutkus merged commit 8580349 into mainJun 12, 2026
1 check was pending
@aurimasbutkus
aurimasbutkus deleted the build-release-fixes branch June 12, 2026 14:36
Sign up for freeto 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

@aurimasbutkus@zygintas