Fix PS1 scripts for Windows compatibility#33679
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds Windows platform support to four PowerShell scripts used for building, deploying, and testing .NET MAUI applications. The changes enable these scripts to handle Windows as a target platform alongside existing support for Android, iOS, and MacCatalyst.
Changes:
- Added "windows" to ValidateSet for Platform parameters across all four scripts
- Implemented Windows-specific build and deployment logic (similar to MacCatalyst approach)
- Updated documentation (synopsis, descriptions, examples) to include Windows platform
- Added PR number auto-detection and markdown reporting features to verify-tests-fail.ps1
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
.github/skills/verify-tests-fail-without-fix/scripts/verify-tests-fail.ps1 |
Added Windows platform support; also includes new PR auto-detection logic and markdown report generation feature |
.github/scripts/shared/Build-AndDeploy.ps1 |
Added Windows build logic (no deploy step, runs on host like MacCatalyst) |
.github/scripts/BuildAndRunSandbox.ps1 |
Added Windows platform configuration and host device handling |
.github/scripts/BuildAndRunHostApp.ps1 |
Added Windows platform configuration, test project mapping, and log handling |
.github/skills/verify-tests-fail-without-fix/scripts/verify-tests-fail.ps1
Outdated
Show resolved
Hide resolved
.github/skills/verify-tests-fail-without-fix/scripts/verify-tests-fail.ps1
Outdated
Show resolved
Hide resolved
kubaflo
reviewed
Jan 22, 2026
.github/skills/verify-tests-fail-without-fix/scripts/verify-tests-fail.ps1
Outdated
Show resolved
Hide resolved
kubaflo
approved these changes
Jan 22, 2026
This was referenced Feb 12, 2026
Merged
Closed
Merged
Merged
Closed
This was referenced Feb 16, 2026
Open
Closed
Open
Merged
Open
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Note
Are you waiting for the changes in this PR to be merged?
It would be very helpful if you could test the resulting artifacts from this PR and let us know in a comment if this change resolves your issue. Thank you!
Description
Updates PowerShell scripts to work correctly on Windows:
Issues Fixed
N/A - Infrastructure improvement