Improvement that comes from ProjectHelper work#51
Conversation
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| param( | ||
| [Parameter(Position = 0)][string]$section | ||
| [Parameter(Position = 0)][string]$section, | ||
| [Parameter()][switch]$Logging |
Check warning
Code scanning / PSScriptAnalyzer
The parameter 'Logging' has been declared but not used. Warning
| $manifest = $Path | Join-Path -child "*.psd1" | Get-Item | Import-PowerShellDataFile | ||
| $requiredModule = $null -eq $manifest.RequiredModules ? @() : $manifest.RequiredModules | ||
|
|
||
| "Found RequiredModules: $($requiredModule.Count)" | Write-Host -ForegroundColor DarkGray |
Check warning
Code scanning / PSScriptAnalyzer
File 'Get-RequiredModule.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
|
|
||
| # Convert to hashtable | ||
| $requiredModule | ForEach-Object{ | ||
| "Processing RequiredModule: $($_| convertto-json -Depth 5)" | Write-Host -ForegroundColor DarkGray |
Check warning
Code scanning / PSScriptAnalyzer
File 'Get-RequiredModule.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
| . SYNOPSIS | ||
| Extracts the required modules from the module manifest | ||
| #> | ||
| function Get-RequiredModule{ |
Check notice
Code scanning / PSScriptAnalyzer
The cmdlet 'Get-RequiredModule' does not have a help comment. Note
| @@ -0,0 +1,28 @@ | |||
| function Test-Result{ | |||
Check notice
Code scanning / PSScriptAnalyzer
The cmdlet 'Test-Result' does not have a help comment. Note test
| @@ -0,0 +1,37 @@ | |||
| function Set-TestName{ | |||
Check notice
Code scanning / PSScriptAnalyzer
The cmdlet 'Set-TestName' does not have a help comment. Note test
| } | ||
| } Export-ModuleMember -Function Set-TestName -Alias st | ||
|
|
||
| function Get-TestName{ |
Check notice
Code scanning / PSScriptAnalyzer
The cmdlet 'Get-TestName' does not have a help comment. Note test
| } | ||
| } Export-ModuleMember -Function Get-TestName -Alias gt | ||
|
|
||
| function Clear-TestName{ |
Check notice
Code scanning / PSScriptAnalyzer
The cmdlet 'Clear-TestName' does not have a help comment. Note test
… list return for non-existent file
…tIncludeSystemFiles
rulasg
force-pushed
the
Improvement-from-projecthelper
branch
from
a709d57 to
e70aeba
Compare
1 participant
Documentation and Developer Guidance
.github/copilot-instructions.md, now tailored forProjectHelper, with detailed explanations of architecture, development patterns, testing conventions, and key gotchas for contributors.Testing and Mocking Infrastructure
Test/include/InvokeMockList.ps1, providing functions to trace and log mock command invocations and their associated files, enhancing test traceability.Test/include/invokeCommand.mock.ps1to use the new tracing functions, improved environment variable naming, and enabled better tracking and exporting of mock command aliases and their usage in tests. [1] [2] [3] [4] [5] [6] [7]Test/public/listIncludes.test.ps1andTest/public/invokeCommand.mock.test.ps1to align with new mocking and tracing mechanisms. [1] [2]CI/CD Workflow Improvements
.github/workflows/deploy_module_on_release.ymland.github/workflows/test_with_TestingHelper.ymlto:Test_Helpermodule for advanced test result validation.Development Environment
.vscode/settings.jsonto simplify terminal configuration.These changes collectively improve developer onboarding, test reliability, and deployment safety for the module.Add plenty of features that come from ProjectHelper work