Skip to content

fix: add checkout step before downloading release assets#28

Merged
luxass merged 1 commit into
mainfrom
fix-homebrew-checkout
May 20, 2026
Merged

fix: add checkout step before downloading release assets#28
luxass merged 1 commit into
mainfrom
fix-homebrew-checkout

Conversation

@luxass

@luxass luxass commented May 20, 2026

Copy link
Copy Markdown
Owner

Add checkout step so gh release download can verify the tag and access the git repo

Summary by CodeRabbit

  • Chores
    • Added a repository checkout step in the Homebrew workflow before downloading release assets.

Review Change Stack

@coderabbitai

coderabbitai Bot commented May 20, 2026

Copy link
Copy Markdown

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: 6c6b0a17-312c-430a-a663-be2f21ade7bb

📥 Commits

Reviewing files that changed from the base of the PR and between a327a7b and 611a40e.

📒 Files selected for processing (1)
  • .github/workflows/reusable-homebrew-tap.yaml

Walkthrough

The pull request adds a workflow step to .github/workflows/reusable-homebrew-tap.yaml that checks out the release repository before downloading release assets. This ensures the repository context is available when subsequent gh release download commands execute in the update-formula job.

Changes

Homebrew Workflow Update

Layer / File(s) Summary
Checkout release repository step
.github/workflows/reusable-homebrew-tap.yaml
A new "checkout release repo" step is added to the update-formula job using actions/checkout@v6.0.2 with persist-credentials: false, positioned before the "download release assets" step.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • luxass/shared-workflows#24: Modifies the same update-formula job in .github/workflows/reusable-homebrew-tap.yaml, specifically updating checkout and GitHub action authentication related to release asset handling.

Poem

🐰 A checkout step hops into place,
Before the assets download with grace,
Repository context, now sure and true,
The workflow path grows stronger too! 🐇

🚥 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: adding a checkout step before downloading release assets, which matches the PR's core objective.
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 fix-homebrew-checkout

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.

@luxass luxass merged commit d590985 into main May 20, 2026
4 checks passed
@luxass luxass deleted the fix-homebrew-checkout branch May 23, 2026 08:32
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