Skip to content

fix(ci): skip goreleaser validate pipe for prefixed tags#577

Merged
tolzhabayev merged 1 commit intomainfrom
fix/goreleaser-skip-validate
May 8, 2026
Merged

fix(ci): skip goreleaser validate pipe for prefixed tags#577
tolzhabayev merged 1 commit intomainfrom
fix/goreleaser-skip-validate

Conversation

@tolzhabayev
Copy link
Copy Markdown
Contributor

Summary

Replaces the release.tag config knob (which only exists in GoReleaser Pro) with --skip=validate, the same pattern the MCP workflow already uses. Also keeps GORELEASER_CURRENT_TAG: \${{ github.ref_name }} so the GitHub release goes to the right (prefixed) tag.

Why

PR #575 tried to keep validation on by setting release.tag: "plugin-validator/v{{ .Version }}" in the goreleaser configs. That field isn't in the OSS schema, so v2.15.4 rejects it:

```
yaml: unmarshal errors:
line 40: field tag not found in type config.Release
```

Failed run after #575 merged

--skip=validate skips the validate pipe (semver parse, working-tree check, template validation) but in a post-tag-push CI run those checks aren't really protecting us — fresh checkout, known-good tag.

Test plan

  • After merge, force-update `plugin-validator/v0.42.0` to the merge commit and re-push to retrigger the publish workflow
  • Confirm publish succeeds and uploads 8 archives + `checksums.txt` to the v0.42.0 release
  • Verify release-please's release notes on v0.42.0 are unchanged
  • Verify Docker Hub push and npm publish complete (downstream of `release-to-github`)

The previous attempt set `release.tag` in the goreleaser configs to map
the bare-semver override back to the prefixed git tag, but `release.tag`
is a GoReleaser Pro field and is rejected by the OSS schema:

    yaml: unmarshal errors:
      line 40: field tag not found in type config.Release

Revert to passing the full prefixed tag through GORELEASER_CURRENT_TAG
and bypassing the offending pipe with `--skip=validate`. Mirrors the
working pattern already used in the MCP workflow.
@github-project-automation github-project-automation Bot moved this from 📬 Triage to 🔬 In review in Grafana Catalog Team May 8, 2026
@tolzhabayev tolzhabayev merged commit bdc6a6e into main May 8, 2026
10 checks passed
@tolzhabayev tolzhabayev deleted the fix/goreleaser-skip-validate branch May 8, 2026 12:20
@github-project-automation github-project-automation Bot moved this from 🔬 In review to 🚀 Shipped in Grafana Catalog Team May 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🚀 Shipped

Development

Successfully merging this pull request may close these issues.

2 participants