Skip to content

Update the CI release action#6

Merged
afollestad merged 2 commits intomainfrom
af/update-release-process
Mar 13, 2026
Merged

Update the CI release action#6
afollestad merged 2 commits intomainfrom
af/update-release-process

Conversation

@afollestad
Copy link
Collaborator

…for latest instructions in block/homebrew-tap.

  • Update the CI release action
  • Re-generate ai-rules files

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6f55b17ef1

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

This comment was marked as resolved.

afollestad and others added 2 commits March 12, 2026 16:02
Co-authored-by: Claude Code <noreply@anthropic.com>
Ai-assisted: true
Co-authored-by: Claude Code <noreply@anthropic.com>
Ai-assisted: true
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.


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

You can also share your feedback on Copilot code review. Take the survey.

-f artifact_url=https://github.com/block/qrgo/releases/download/${{ github.ref_name }}/qrgo-release.tar.gz \
-f sha256=${{ steps.generate_hash.outputs.file_hash }}
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} No newline at end of file
Copy link

Copilot AI Mar 12, 2026

Choose a reason for hiding this comment

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

The gh workflow run --repo block/homebrew-tap call is authenticated with secrets.GITHUB_TOKEN, which is scoped to this repository and typically cannot trigger workflows in a different repo. Use a dedicated secret (PAT or GitHub App token) that has permission to run workflows in block/homebrew-tap, and pass that token to gh for this step.

Suggested change
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
GITHUB_TOKEN: ${{ secrets.HOMEBREW_TAP_WORKFLOW_TOKEN }}

Copilot uses AI. Check for mistakes.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This is likely true as it was before - not yet sure how to fix this though

Choose a reason for hiding this comment

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

Yep, it's still true. To fix it, we'll need to set up organization secrets:

  1. Homebrew Tap Github App and
  2. a private key

and then add these secrets to each repo. We should be able to copy what our internal tap uses, but I don't have permission to see that.

If you'd like to fix this yourself, I can add you as a homebrew-tap maintainer and you can post in the internal #open-source channel.

Copy link
Collaborator Author

@afollestad afollestad Mar 12, 2026

Choose a reason for hiding this comment

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

@tcmulcahy I think I already have access to homebrew-tap; just asked in #opensource about getting a shared app/secret

@afollestad afollestad merged commit 8aa2d6c into main Mar 13, 2026
10 checks passed
@afollestad afollestad deleted the af/update-release-process branch March 13, 2026 00:17
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.

3 participants