Skip to content

fix: update artifact type for goreleaser v2#302

Merged
retr0h merged 1 commit intomainfrom
fix-goreleaser-v2-artifact-type
Jan 29, 2026
Merged

fix: update artifact type for goreleaser v2#302
retr0h merged 1 commit intomainfrom
fix-goreleaser-v2-artifact-type

Conversation

@retr0h
Copy link
Owner

@retr0h retr0h commented Jan 29, 2026

GoReleaser v2 renamed the artifact type from "Brew Tap" to "Homebrew Formula", causing the wheel build to fail with a KeyError when looking for description/license metadata.

GoReleaser v2 renamed the artifact type from "Brew Tap" to
"Homebrew Formula", causing the wheel build to fail with a
KeyError when looking for description/license metadata.
@retr0h
Copy link
Owner Author

retr0h commented Jan 29, 2026

@0xDEC0DE can you have a look?

@retr0h retr0h merged commit ea00ae6 into main Jan 29, 2026
10 checks passed
@github-actions github-actions bot deleted the fix-goreleaser-v2-artifact-type branch January 29, 2026 22:00
Copy link
Contributor

@0xDEC0DE 0xDEC0DE left a comment

Choose a reason for hiding this comment

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

Looks like it builds

# Burn a pass through the list to steal some useful bits from the Brew config
for artifact in self.artifacts:
if artifact["type"] == "Brew Tap":
if artifact["type"] == "Homebrew Formula":
Copy link
Contributor

@0xDEC0DE 0xDEC0DE Jan 29, 2026

Choose a reason for hiding this comment

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

This has always in a sense been cheating, and "stealing" info from the GoReleaser configs, so that you could have to update them in more than one place.

Originally, I avoided using the internal_type item in the JSON under the auspices of avoiding internal implementation details that might randomly change out from under me, but it's beginning to look like those are in fact some of the most stable items in here. Might want to switch to using them instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants