Skip to content

ci: call publish workflow from draft when creating pre-release#106

Merged
bcheidemann merged 3 commits into
emmanuelnk:mainfrom
bcheidemann:ci/fix-publish-beta-releases
Feb 16, 2026
Merged

ci: call publish workflow from draft when creating pre-release#106
bcheidemann merged 3 commits into
emmanuelnk:mainfrom
bcheidemann:ci/fix-publish-beta-releases

Conversation

@bcheidemann
Copy link
Copy Markdown
Collaborator

This is a follow on PR from #105.

Turns out, I did a derp. I thought I'd tested that the GH release created by release-drafter would trigger the publish workflow, but I didn't, and it doesn't. This is expected behavior as per the GH docs:

When you use the repository's GITHUB_TOKEN to perform tasks, events triggered by the GITHUB_TOKEN, with the exception of workflow_dispatch and repository_dispatch, will not create a new workflow run. This prevents you from accidentally creating recursive workflow runs.

A solution (the one implemented in this PR) is to explicitly call the publish workflow from the draft workflow.

@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 13, 2026

@bcheidemann is attempting to deploy a commit to the emmanuelnk's projects Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 15, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
github-actions-workflow-ts Ready Ready Preview, Comment Feb 15, 2026 5:20pm

@bcheidemann bcheidemann merged commit d608cde into emmanuelnk:main Feb 16, 2026
11 checks passed
@bcheidemann
Copy link
Copy Markdown
Collaborator Author

bcheidemann commented Feb 16, 2026

@emmanuelnk looks like this is working now =) https://github.com/emmanuelnk/github-actions-workflow-ts/actions/runs/22057952159
Screenshot from 2026-02-16 10-10-48

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.

2 participants