Skip to content

Revert "Add PowerShell dependency to winget manifests"#1548

Merged
devm33 merged 1 commit intomainfrom
revert-1497-add-powershell-dependency
Feb 19, 2026
Merged

Revert "Add PowerShell dependency to winget manifests"#1548
devm33 merged 1 commit intomainfrom
revert-1497-add-powershell-dependency

Conversation

@devm33
Copy link
Member

@devm33 devm33 commented Feb 19, 2026

Copilot AI review requested due to automatic review settings February 19, 2026 19:59
Copy link
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 reverts changes from PR #1497 that added PowerShell dependency manipulation to the winget package submission workflow. The revert was triggered by a failing GitHub Actions run (https://github.com/github/copilot-cli/actions/runs/22186218770), restoring the workflow to its simpler, original state where wingetcreate directly submits the package without manual manifest modification.

Changes:

  • Removed PowerShell script logic that added Microsoft.PowerShell dependency to installer manifests
  • Restored direct --submit flag to wingetcreate instead of two-step --out + submit process

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

@devm33 devm33 merged commit befad01 into main Feb 19, 2026
8 checks passed
@devm33 devm33 deleted the revert-1497-add-powershell-dependency branch February 19, 2026 22:06
SteveSandersonMS added a commit that referenced this pull request Feb 19, 2026
Re-applies the change from PR #1497 which was reverted in PR #1548.

The original broke the workflow YAML because the PowerShell here-string
content and closing tag had zero indentation, which terminated the YAML
literal block scalar (run: |). Fixed by indenting the here-string body
to match the block indentation level; YAML strips the leading spaces,
leaving valid PowerShell with the closing tag at column 0.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
SteveSandersonMS added a commit that referenced this pull request Feb 20, 2026
Re-applies the change from PR #1497 which was reverted in PR #1548.

The original broke the workflow YAML because the PowerShell here-string
content and closing tag had zero indentation, which terminated the YAML
literal block scalar (run: |). Fixed by indenting the here-string body
to match the block indentation level; YAML strips the leading spaces,
leaving valid PowerShell with the closing tag at column 0.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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

Comments