Skip to content
This repository was archived by the owner on May 21, 2026. It is now read-only.

Fix release workflow auth (drop setup-homebrew)#2

Closed
sgerlach wants to merge 1 commit into
mainfrom
fix/update-formula-auth
Closed

Fix release workflow auth (drop setup-homebrew)#2
sgerlach wants to merge 1 commit into
mainfrom
fix/update-formula-auth

Conversation

@sgerlach
Copy link
Copy Markdown
Contributor

Summary

First manual dispatch of update-formula.yml (for v0.6.1) failed at
the push step:

fatal: could not read Username for 'https://github.com': No such device or address

Homebrew/actions/setup-homebrew@master rewrites the repo's git config,
wiping the http.https://github.com/.extraheader that
actions/checkout@v4 sets from GITHUB_TOKEN. The local commit succeeds,
but git push has nothing to authenticate with.

The renderer only needs curl, shasum, sed, and awk — all of which
are preinstalled on ubuntu-latest. Drop setup-homebrew from this
workflow. test.yml keeps it (it genuinely needs brew for audit/install/test).

Test plan

  • After merge, run gh workflow run update-formula.yml -f version=0.6.1
    and confirm it commits Formula/hawkop.rb on main.
  • brew untap stackhawk/hawkop; brew tap stackhawk/hawkop; brew install hawkop
    installs successfully.

Homebrew/actions/setup-homebrew resets the checked-out repo's git config,
wiping the auth extraheader that actions/checkout@v4 configures from
GITHUB_TOKEN. The 'Commit and push' step then fails with
'could not read Username for https://github.com'.

The renderer only needs curl, shasum, sed, and awk — all preinstalled on
ubuntu-latest. Drop the setup-homebrew step. The PR test workflow still
keeps setup-homebrew (it needs brew for audit/install/test).
@sgerlach
Copy link
Copy Markdown
Contributor Author

Closing — we're mirroring hawkscan's homebrew-cli pattern instead (dumb tap, release pipeline in hawkop pushes directly). Opening a cleanup PR that removes the workflow entirely.

@sgerlach sgerlach closed this Apr 22, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant