Skip to content

fix(release): remove invalid web workflow dispatch#2212

Open
chaliy wants to merge 1 commit into
mainfrom
2026-07-26-fix-release-workflow-dispatch-issue
Open

fix(release): remove invalid web workflow dispatch#2212
chaliy wants to merge 1 commit into
mainfrom
2026-07-26-fix-release-workflow-dispatch-issue

Conversation

@chaliy

@chaliy chaliy commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

Motivation

  • The release job attempted to run a nonexistent workflow publish-web.yml, which causes gh workflow run to exit non-zero and abort subsequent release steps.
  • The browser publish is already handled by publish-wasm.yml, so the extra dispatch is a duplicate and a regression.

Description

  • Remove the invalid dispatch line gh workflow run publish-web.yml --ref "$TAG" from .github/workflows/release.yml.
  • Update scripts/tests/test_release_workflow.py to expect publish-wasm.yml instead of publish-web.yml and add test_release_dispatch_targets_exist to assert each gh workflow run target exists under .github/workflows.
  • Keep existing browser/WASM publishing behavior intact by relying on publish-wasm.yml.

Testing

  • Reproduced the failure with python3 -m unittest scripts.tests.test_release_workflow before the fix and observed the missing publish-web.yml assertion fail.
  • Ran python3 -m unittest discover -s scripts/tests -p 'test_*.py' after the fix and all 8 tests passed.
  • Ran ruff check/ruff format --check on the modified test file and git diff --check with no issues.

Codex Task

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 26, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
bashkit 07ba78f Commit Preview URL Jul 26 2026, 10:45 PM

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant