Commit 22e2ea2
fix: release workflow goreleaser --skip=validate — accept ui_dist drift at release time (#18)
v1.0.3 release run surfaced third bug after npm-auth + cp-cwd fixes landed:
release failed after 0s
error= git is in a dirty state
M internal/ui_dist/index.html
Root cause: Build-admin-UI step rebuilds internal/ui_dist/ from workflow
repo HEAD (latest UI source), which may differ from this repo's committed
snapshot (kept in-tree for go:embed at dev time). GoReleaser's
git-dirty-state validation correctly flags this as 'modified files since
last commit'.
The fresh build IS the intended release artifact — we WANT the just-built
dist embedded in the release binary, not the stale committed snapshot. So
the dirty state is expected at release time.
Fix: --skip=validate flag bypasses dirty check. Per goreleaser docs this
is the appropriate escape hatch when build artifacts intentionally differ
from committed state.
Followup: cut v1.0.4 from main after this lands.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>1 parent 2e26487 commit 22e2ea2
1 file changed
Lines changed: 9 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
55 | 55 | | |
56 | 56 | | |
57 | 57 | | |
58 | | - | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
59 | 67 | | |
60 | 68 | | |
61 | 69 | | |
| |||
0 commit comments