Conversation
Co-authored-by: Stensel8 <102481635+Stensel8@users.noreply.github.com>
Clean up bug report template.
…ting Create bug report issue template for WinDeploy
|
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation. |
There was a problem hiding this comment.
Pull Request Overview
This release (v0.1.2) standardizes script documentation and improves path detection for remote execution scenarios. Key changes include streamlining documentation blocks across all scripts, enhancing the startup banner to display execution context (local/GitHub/remote), and improving error handling in empty catch blocks.
- Standardized script headers and documentation blocks for consistency
- Enhanced remote execution detection and display in startup banner
- Improved error logging in catch blocks and path resolution
- Added safety warning for driver import operations
Reviewed Changes
Copilot reviewed 33 out of 33 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| autounattend.xml | New unattended Windows installation configuration file |
| VERSION | Version bump to v0.1.2 |
| Scripts/Utilities/*.psm1 | Added UTF-8 BOM to all module files |
| Scripts/Start.ps1 | Enhanced path detection, improved banner display, added error logging |
| Scripts/*.ps1 | Standardized headers, streamlined documentation blocks |
| Scripts/Import-Drivers.ps1 | Added user confirmation warning for driver installation |
| Scripts/Install-WindowsUpdates.ps1 | Fixed parameter naming in background job |
| README.md | Enhanced project structure documentation with [AUTO]/[UTIL] tags |
| CHANGELOG.md | Added v0.1.2 release notes |
| .github/ISSUE_TEMPLATE/bug_report.md | New bug report template |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 33 out of 33 changed files in this pull request and generated no new comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 33 out of 33 changed files in this pull request and generated no new comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
|
||
| # Get Win32 apps from registry | ||
| Write-Log "Scanning registry for Win32 applications..." -Level Info | ||
| Write-Host "Scanning registry for Win32 applications..." -ForegroundColor Cyan |
Check warning
Code scanning / PSScriptAnalyzer
File 'Get-InstalledSoftware.ps1' uses Write-Host. Avoid using Write-Host because it might not work in all hosts, does not work when there is no host, and (prior to PS 5.0) cannot be suppressed, captured, or redirected. Instead, use Write-Output, Write-Verbose, or Write-Information. Warning
|
|
||
| $allSoftware += $win32Apps | ||
| Write-Log "Found $($win32Apps.Count) Win32 applications" -Level Info | ||
| Write-Host "Found $($win32Apps.Count) Win32 applications" -ForegroundColor Green |
Check warning
Code scanning / PSScriptAnalyzer
File 'Get-InstalledSoftware.ps1' uses Write-Host. Avoid using Write-Host because it might not work in all hosts, does not work when there is no host, and (prior to PS 5.0) cannot be suppressed, captured, or redirected. Instead, use Write-Output, Write-Verbose, or Write-Information. Warning
|
|
||
| # Get Store/UWP apps | ||
| Write-Log "Scanning for Store/UWP applications..." -Level Info | ||
| Write-Host "Scanning for Store/UWP applications..." -ForegroundColor Cyan |
Check warning
Code scanning / PSScriptAnalyzer
File 'Get-InstalledSoftware.ps1' uses Write-Host. Avoid using Write-Host because it might not work in all hosts, does not work when there is no host, and (prior to PS 5.0) cannot be suppressed, captured, or redirected. Instead, use Write-Output, Write-Verbose, or Write-Information. Warning
|
|
||
| $allSoftware += $storeApps | ||
| Write-Log "Found $($storeApps.Count) Store applications" -Level Info | ||
| Write-Host "Found $($storeApps.Count) Store applications" -ForegroundColor Green |
Check warning
Code scanning / PSScriptAnalyzer
File 'Get-InstalledSoftware.ps1' uses Write-Host. Avoid using Write-Host because it might not work in all hosts, does not work when there is no host, and (prior to PS 5.0) cannot be suppressed, captured, or redirected. Instead, use Write-Output, Write-Verbose, or Write-Information. Warning
| $allSoftware = $allSoftware | Sort-Object Name, Version -Unique | ||
|
|
||
| Write-Log "Total unique applications: $($allSoftware.Count)" -Level Success | ||
| Write-Host "Total unique applications: $($allSoftware.Count)" -ForegroundColor Green |
Check warning
Code scanning / PSScriptAnalyzer
File 'Get-InstalledSoftware.ps1' uses Write-Host. Avoid using Write-Host because it might not work in all hosts, does not work when there is no host, and (prior to PS 5.0) cannot be suppressed, captured, or redirected. Instead, use Write-Output, Write-Verbose, or Write-Information. Warning
| Write-Log "Error: $_" -Level Error | ||
| } finally { | ||
| Complete-Script | ||
| Write-Host "Error: $_" -ForegroundColor Red |
Check warning
Code scanning / PSScriptAnalyzer
File 'Get-InstalledSoftware.ps1' uses Write-Host. Avoid using Write-Host because it might not work in all hosts, does not work when there is no host, and (prior to PS 5.0) cannot be suppressed, captured, or redirected. Instead, use Write-Output, Write-Verbose, or Write-Information. Warning
## [0.5.0] - 2025-11-14 ### Added - Added documentation for Intune Autopilot device preparation setup (`Docs/Intune-Autopilot-Setup.md`) - Added RMM agent installation support with USB detection and download fallback ### Changed - Simplified and improved project structure for better maintainability - Streamlined deployment scripts with cleaner, more maintainable code - Updated README.md with comprehensive documentation and updated flowchart - Reorganized scripts into `Scripts/Deployment/` for better organization ### Removed - Removed complex modular architecture in favor of inline scripts - Removed unused utility modules and scripts ### Fixed - Improved error handling and logging across all scripts - Enhanced compatibility and reliability of deployment process
## [0.5.0] - 2025-11-14 ### Added - Added documentation for Intune Autopilot device preparation setup (`Docs/Intune-Autopilot-Setup.md`) - Added RMM agent installation support with USB detection and download fallback ### Changed - Simplified and improved project structure for better maintainability - Streamlined deployment scripts with cleaner, more maintainable code - Updated README.md with comprehensive documentation and updated flowchart - Reorganized scripts into `Scripts/Deployment/` for better organization ### Removed - Removed complex modular architecture in favor of inline scripts - Removed unused utility modules and scripts ### Fixed - Improved error handling and logging across all scripts - Enhanced compatibility and reliability of deployment process
[0.5.0] - 2025-11-14
Added
Docs/Intune-Autopilot-Setup.md)Changed
Scripts/Deployment/for better organizationRemoved
Fixed