Skip to content

refactor(workflow): replace isPrerelease with force input#186

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

refactor(workflow): replace isPrerelease with force input#186
HeyItsGilbert merged 3 commits into
mainfrom
refactor/publish-workflow-inputs

Conversation

@HeyItsGilbert
Copy link
Copy Markdown
Member

Summary

Aligns with PowerShellOrg/.github#12.

  • Removes isPrerelease dispatch input and with: pass-through — the reusable workflow now infers prerelease from PSData.Prerelease in the manifest automatically
  • Adds force dispatch input to bypass the PSGallery existence check when re-triggering a failed job

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

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.

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

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 30, 2026 16:13
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the publish workflow inputs to align with the reusable PowerShell release workflow by replacing the prerelease flag with a force-publish option.

Changes:

  • Removes the isPrerelease workflow input and pass-through.
  • Adds a force workflow input and passes it to the reusable release workflow.
  • Documents the intended failed-publish re-trigger pattern.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/publish.yml
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>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread .github/workflows/publish.yml Outdated
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>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@HeyItsGilbert HeyItsGilbert merged commit e7cb777 into main May 30, 2026
6 checks passed
@HeyItsGilbert HeyItsGilbert deleted the refactor/publish-workflow-inputs branch May 30, 2026 16:20
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