Skip to content

fix: trigger release workflow on release published event#27

Merged
jim80net merged 2 commits intomainfrom
fix/release-workflow-trigger
Mar 16, 2026
Merged

fix: trigger release workflow on release published event#27
jim80net merged 2 commits intomainfrom
fix/release-workflow-trigger

Conversation

@jim80net
Copy link
Owner

Summary

  • Switch release workflow trigger from on: push: tags: 'v*' to on: release: types: [published]
  • Update checkout and upload steps to use github.event.release.tag_name instead of github.ref_name
  • Also includes remaining skill-router → memex-claude naming cleanup in CONTRIBUTING.md, .gitignore, and sharp stubs

Why

release-please creates tags via the GitHub API using the default GITHUB_TOKEN, which doesn't fire on: push: tags workflows. This is why v1.3.0 has no binary assets.

Test plan

  • Merge this PR
  • Merge the resulting release-please PR (v1.3.1)
  • Verify the Release workflow triggers and uploads binaries
  • Verify bin/install.sh successfully downloads the binary

🤖 Generated with Claude Code

jim80net and others added 2 commits March 15, 2026 19:42
CONTRIBUTING.md, .gitignore, and sharp stubs still had old naming
from before the rebrand in PR #24.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
release-please creates tags via the GitHub API, which doesn't fire
`on: push: tags` workflows when using the default GITHUB_TOKEN.
Switch to `on: release: types: [published]` so binary builds run
automatically after release-please creates a release.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jim80net jim80net merged commit 491b901 into main Mar 16, 2026
5 checks passed
@claude
Copy link

claude bot commented Mar 16, 2026

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant