Skip to content

Remove --rm-dist from goreleaser action args#25

Merged
noborus merged 1 commit into
mainfrom
goreleaser-opt
May 22, 2026
Merged

Remove --rm-dist from goreleaser action args#25
noborus merged 1 commit into
mainfrom
goreleaser-opt

Conversation

@noborus
Copy link
Copy Markdown
Owner

@noborus noborus commented May 22, 2026

Summary by CodeRabbit

  • Chores
    • Updated release workflow configuration to optimize the build process.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 22, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c2834599-0574-4f03-b2cf-bc5dca5873d2

📥 Commits

Reviewing files that changed from the base of the PR and between b64aad9 and 30c0063.

📒 Files selected for processing (1)
  • .github/workflows/release.yml

📝 Walkthrough

Walkthrough

The GoReleaser action in the release workflow is simplified by removing the --rm-dist flag from its arguments, changing from args: release --rm-dist to args: release on line 22 of .github/workflows/release.yml.

Changes

Release Workflow Configuration

Layer / File(s) Summary
GoReleaser action arguments
.github/workflows/release.yml
The --rm-dist flag is removed from GoReleaser's action arguments, simplifying the release invocation from release --rm-dist to release.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A flag flies away on the wind so free,
--rm-dist departs from CI's decree,
The release flows cleaner, simpler than before,
One tiny change opens a lighter door! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: removing the --rm-dist flag from the GoReleaser action arguments.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check 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 goreleaser-opt

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.

@noborus noborus merged commit e398eae into main May 22, 2026
2 checks passed
@noborus noborus deleted the goreleaser-opt branch May 22, 2026 03:52
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