ci: fix release workflow producing no tags on push to main#4
Merged
Conversation
The edge tag was gated on {{is_default_branch}}, but this repo's default
branch isn't main, so pushes to main generated zero tags and buildx failed
with "tag is needed when pushing to registry". Gate edge on an explicit
github.ref == refs/heads/main check (and latest on tag pushes), and set
flavor: latest=false so the only `latest` is our explicit one.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013Lj6nYJQDtLaZFvvEQJGM4
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes the failed publish run (#1).
Cause: the
edgetag was gated on{{is_default_branch}}. This repo's default branch is stillclaude/youthful-pasteur-2f6s72(the first branch pushed), notmain— so a push tomainevaluatedis_default_branchas false, the metadata-action produced zero tags, and buildx failed withtag is needed when pushing to registry. (Login/QEMU/buildx/image-name all worked; only the tag list was empty.)Fix: gate
edgeon an explicitgithub.ref == 'refs/heads/main'andlateston tag pushes, and setflavor: latest=false. On merge this should produce theedgetag and actually run the multi-arch build (the first real test of thebetter-sqlite3Alpine build fix).Recommended follow-up (one-click, GitHub side): set the repo's default branch to
mainso PR bases and CI default-branch logic behave as expected.🤖 Generated with Claude Code
Generated by Claude Code