Skip to content

Remove redundant PS scripts#408

Merged
MGudgin merged 1 commit into
mainfrom
user/gudge/remove_redundant_ps_scripts
May 27, 2026
Merged

Remove redundant PS scripts#408
MGudgin merged 1 commit into
mainfrom
user/gudge/remove_redundant_ps_scripts

Conversation

@MGudgin

@MGudgin MGudgin commented May 23, 2026

Copy link
Copy Markdown
Member

This PR removes a couple of Powershell scripts introduced by PR #389 that are
not needed and fixes up docs/code comments that referenced them.

Microsoft Reviewers: Open in CodeFlow

Copilot AI review requested due to automatic review settings May 23, 2026 21:23

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 removes redundant PowerShell host-prep helper scripts that duplicated wxc-exec --prepare-system-drive / --unprepare-system-drive behavior, and begins cleaning up documentation references to those scripts.

Changes:

  • Deletes the prepare/unprepare PowerShell helper scripts under scripts/host-prep/.
  • Removes one obsolete mention of PowerShell script semantics from docs/host-prep.md.
Show a summary per file
File Description
scripts/host-prep/Prepare-SystemDriveForAppContainer.ps1 Removes the redundant prepare helper script.
scripts/host-prep/Unprepare-SystemDriveForAppContainer.ps1 Removes the redundant unprepare helper script.
docs/host-prep.md Removes one reference tying precise revoke behavior to the deleted script.

Copilot's findings

  • Files reviewed: 3/3 changed files
  • Comments generated: 1

Comment thread docs/host-prep.md
@MGudgin MGudgin force-pushed the user/gudge/remove_redundant_ps_scripts branch from d1f294e to 49d91d4 Compare May 23, 2026 21:30
@kanismohammed kanismohammed requested a review from a team May 26, 2026 18:44
This PR removes a couple of Powershell scripts introduced by PR #389 that are
not needed and fixes up docs/code comments that referenced them.
@MGudgin MGudgin force-pushed the user/gudge/remove_redundant_ps_scripts branch from 49d91d4 to 2aad43e Compare May 27, 2026 03:05
@MGudgin MGudgin merged commit 3a37b39 into main May 27, 2026
20 checks passed
@MGudgin MGudgin deleted the user/gudge/remove_redundant_ps_scripts branch May 27, 2026 19: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.

3 participants