Normalize PowerShell line endings in VM support changes - #4
Conversation
Co-authored-by: vartaxe <11176349+vartaxe@users.noreply.github.com>
There was a problem hiding this comment.
🟡 Changes recommended
The file still contains mixed line endings (CRLF and LF), so the intended whitespace/normalization cleanup is not yet consistently applied and may continue to cause diffs/whitespace-check noise.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
This PR aims to clean up line-ending issues introduced in recent virtual-machine support additions to Invoke-CMApplyDriverPackage.ps1, so Git no longer reports whitespace problems while keeping the script’s behavior unchanged.
Changes:
- Normalizes formatting/line-ending-related diffs across the recently added VM support sections (help examples,
-AllowVirtualMachineparameter block, VM detection/filtering helpers, and version log line). - Keeps functional script content the same while limiting the scope of the rewrite.
File summaries
| File | Description |
|---|---|
| Invoke-CMApplyDriverPackage.ps1 | Line-ending/whitespace normalization in the VM support-related additions and nearby lines. |
Review details
Copilot wasn't able to review any files in this pull request.
- Files reviewed: 0/1 changed files
- Comments generated: 1
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| # Enable TLS 1.2 support for downloading modules from PSGallery | ||
| [Net.ServicePointManager]::SecurityProtocol = [Net.SecurityProtocolType]::Tls12 | ||
| $Script:IsVirtualMachine = $false | ||
| $Script:VirtualMachineModels = @("Virtual Machine", "VMware Virtual Platform", "VirtualBox", "HVM domU", "KVM", "VMware7,1") | ||
| $Script:VirtualMachinePackagePattern = "\b(virtual machine|vmware|vmxnet|pvscsi|hyper[- ]?v|parallels|virtualbox|virtio|kvm|xen)\b" | ||
| $Script:IsVirtualMachine = $false | ||
| $Script:VirtualMachineModels = @("Virtual Machine", "VMware Virtual Platform", "VirtualBox", "HVM domU", "KVM", "VMware7,1") | ||
| $Script:VirtualMachinePackagePattern = "\b(virtual machine|vmware|vmxnet|pvscsi|hyper[- ]?v|parallels|virtualbox|virtio|kvm|xen)\b" |
|
@copilot Fix the code for all comments in this review thread. When a review comment includes a suggested change, apply the suggestion exactly. Do not make changes beyond what is described in the linked review thread. |
Recent virtual-machine support additions used CRLF endings that Git interpreted as trailing whitespace.
Invoke-CMApplyDriverPackage.ps1.