Skip to content

Commit 8d0059c

Browse files
tablackburnclaude
andauthored
refactor(build): drop $_ prefix from helper variable names (#20)
Rename $_moduleVersion → $moduleVersion and $_stagedOutput → $stagedOutput in build.psake.ps1's properties block. Pure rename, no behavior change. The $_ prefix was meant to signal "internal helper, not configuration" but is easy to misread as the $_ automatic pipeline variable. The shorter names are clearer in context (assignment targets in a properties{} block, not script-block parameters). Same rename applied across the user's downstream modules (JsmOperations, PlexAutomationToolkit, ReScenePS) so the template and all initialized modules stay in sync. Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 03b9864 commit 8d0059c

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

build.psake.ps1

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -22,11 +22,11 @@ properties {
2222
if (-not $Env:BHPSModuleManifest -or -not $Env:BHProjectName) {
2323
throw 'Coverage configuration requires BuildHelpers env vars. Run via ./build.ps1 or call Set-BuildEnvironment first.'
2424
}
25-
$_moduleVersion = (Import-PowerShellDataFile -Path $Env:BHPSModuleManifest).ModuleVersion
26-
$_stagedOutput = [IO.Path]::Combine($PSScriptRoot, 'Output', $Env:BHProjectName, $_moduleVersion)
25+
$moduleVersion = (Import-PowerShellDataFile -Path $Env:BHPSModuleManifest).ModuleVersion
26+
$stagedOutput = [IO.Path]::Combine($PSScriptRoot, 'Output', $Env:BHProjectName, $moduleVersion)
2727
$PSBPreference.Test.CodeCoverage.Files = @(
28-
"$_stagedOutput/Public/*.ps1"
29-
"$_stagedOutput/Private/*.ps1"
28+
"$stagedOutput/Public/*.ps1"
29+
"$stagedOutput/Private/*.ps1"
3030
)
3131
$PSBPreference.Test.CodeCoverage.Threshold = 0 # Threshold enforced by Codecov
3232
$PSBPreference.Test.CodeCoverage.OutputFile = [IO.Path]::Combine($PSScriptRoot, 'out', 'codeCoverage.xml')

0 commit comments

Comments
 (0)