Skip to content

Fix Zapstore publish workflow flag handling#483

Merged
AnthonyRonning merged 1 commit intomasterfrom
fix/zapstore-publish-ci
Apr 21, 2026
Merged

Fix Zapstore publish workflow flag handling#483
AnthonyRonning merged 1 commit intomasterfrom
fix/zapstore-publish-ci

Conversation

@AnthonyRonning
Copy link
Copy Markdown
Contributor

@AnthonyRonning AnthonyRonning commented Apr 21, 2026

Summary

  • replace the removed �[1m-y�[0m flag with �[1m--quiet�[0m in the Zapstore publish workflow
  • fail the workflow if �[1mzsp�[0m prints usage text or an unknown-flag error while still exiting successfully
  • preserve normal success and no-op cases for Zapstore publishes

Validation

  • git diff --check
  • prettier --check .github/workflows/zapstore-publish.yml
  • simulated the workflow shell logic for help-text false-success, success, no-op, and failure cases
  • pre-commit hook: frontend build and tests

Open in Devin Review

Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying maple with  Cloudflare Pages  Cloudflare Pages

Latest commit: c49933c
Status: ✅  Deploy successful!
Preview URL: https://03da6c33.maple-ca8.pages.dev
Branch Preview URL: https://fix-zapstore-publish-ci.maple-ca8.pages.dev

View logs

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

@AnthonyRonning AnthonyRonning merged commit a11f0cc into master Apr 21, 2026
12 checks passed
@AnthonyRonning AnthonyRonning deleted the fix/zapstore-publish-ci branch April 21, 2026 02:54
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