Skip to content

refactor: dogfood setup-pnpm, pr-status-comment, trigger-website-update - #13

Merged
baxyz merged 2 commits into
mainfrom
feat/dogfood-new-actions
Jul 28, 2026
Merged

refactor: dogfood setup-pnpm, pr-status-comment, trigger-website-update#13
baxyz merged 2 commits into
mainfrom
feat/dogfood-new-actions

Conversation

@baxyz

@baxyz baxyz commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Stacked on feat: add setup-pnpm, pr-status-comment, trigger-website-update actions #12 (adds the 3 new composite actions) — CI on this PR will show red until feat: add setup-pnpm, pr-status-comment, trigger-website-update actions #12 merges to main, since these workflows reference helpers4/action/<name>@main, which only resolves once feat: add setup-pnpm, pr-status-comment, trigger-website-update actions #12 is merged. This is expected, not a bug.
  • pr-validation.yml:
    • shellcheck job now uses ludeeus/action-shellcheck (350★, already does this) instead of a hand-rolled apt-get + shellcheck sequence
    • The ~90-line status-comment actions/github-script block is replaced by one call to helpers4/action/pr-status-comment@main
    • Every job's "Set status" step + outputs: status: ... block is deleted — needs.<job>.result already exposes success/failure/skipped/cancelled natively, no custom output plumbing was ever needed
    • Net: 160 lines removed from this file alone
  • release.yml: the Trigganator token + repository-dispatch steps collapse into one helpers4/action/trigger-website-update@main call (same continue-on-error: true, same non-blocking behavior as before)
  • setup-pnpm isn't dogfooded here — this repo has no Node/pnpm build step of its own (bash-only actions), so there's nothing in this repo's CI for it to replace. It'll get its first real exercise when rolled out to typescript/website in a later pass (tracked in TODO.md).

Test plan

🤖 Generated with Claude Code

@baxyz
baxyz changed the base branch from feat/new-composite-actions to main July 28, 2026 20:24
@baxyz
baxyz force-pushed the feat/dogfood-new-actions branch 2 times, most recently from 0e0a42e to 4991f76 Compare July 28, 2026 21:13
baxyz and others added 2 commits July 28, 2026 21:22
…ite-update

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…mers

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@baxyz
baxyz force-pushed the feat/dogfood-new-actions branch from 618d602 to 9f718af Compare July 28, 2026 21:22
@github-actions

Copy link
Copy Markdown

✅ PR Validation Passed

All checks passed!


📋 Pipeline Status

Job Status
🧾 Conventional Commits passing
🐚 ShellCheck passing
🧪 Action Tests passing
📄 Manifest Validation passing
📦 setup-pnpm Tests passing
💬 pr-status-comment Tests passing

🤖 Generated by @helpers4 CI • 2026-07-28

@baxyz
baxyz merged commit 215739f into main Jul 28, 2026
9 checks passed
@baxyz
baxyz deleted the feat/dogfood-new-actions branch July 28, 2026 21:32
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