Skip to content

refactor(workflow): add dispatch inputs and replace isPrerelease with force#468

Merged
HeyItsGilbert merged 3 commits into
mainfrom
refactor/publish-workflow-inputs
May 30, 2026
Merged

refactor(workflow): add dispatch inputs and replace isPrerelease with force#468
HeyItsGilbert merged 3 commits into
mainfrom
refactor/publish-workflow-inputs

Conversation

@HeyItsGilbert
Copy link
Copy Markdown
Member

Summary

Aligns with PowerShellOrg/.github#12.

  • Adds version, force, and dry_run to workflow_dispatch inputs (previously no manual inputs were exposed)
  • force bypasses the PSGallery existence check when re-triggering a failed job — previously there was no way to do this without a code change

Re-trigger pattern

If create_release succeeds but publish fails (e.g. expired secret), re-trigger with:
force=true, create_release=false, publish=true

🤖 Generated with Claude Code

HeyItsGilbert and others added 3 commits May 30, 2026 09:12
… force

Aligns with PowerShellOrg/.github#12 which removes isPrerelease (now
inferred from PSData.Prerelease in the manifest) and adds force to
bypass the PSGallery existence check when re-triggering a failed job.

Also exposes version and dry_run via workflow_dispatch for manual runs.

Re-trigger pattern: force=true, create_release=false, publish=true

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Required to actually use the force re-trigger pattern documented in the
previous commit. Without these inputs, operators could not set
create_release=false when re-triggering a failed publish job.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
inputs context is empty on non-dispatch triggers; the previous
`!= false` expression evaluated empty as equal to false under
GitHub Actions loose comparison, skipping both jobs on push.

Use `github.event_name != 'workflow_dispatch' || inputs.X` instead:
short-circuits to true on push, falls through to the explicit input
value on dispatch.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

Test Results

  3 files  213 suites   32s ⏱️
243 tests 241 ✅ 2 💤 0 ❌
729 runs  723 ✅ 6 💤 0 ❌

Results for commit abefe57.

@HeyItsGilbert HeyItsGilbert merged commit ef24468 into main May 30, 2026
6 of 7 checks passed
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