fix(PipelineExecution): add ShouldProcess check to Set-DefaultParameterValue function

MariusStorhaug · MariusStorhaug · commit 6373290af053 · 2026-05-24T10:09:56.000+02:00
diff --git a/guidance/PipelineExecution.ps1 b/guidance/PipelineExecution.ps1
@@ -11,9 +11,11 @@
     param (
         [string]$Parameter = 'Default-Parameter'
     )
-    Write-Verbose "Set-DefaultParameterValue: Setting Parameter to '$Parameter'"
-    Start-Sleep -Seconds 1
-    return $Parameter
+    if ($PSCmdlet.ShouldProcess($Parameter, 'Set default parameter value')) {
+        Write-Verbose "Set-DefaultParameterValue: Setting Parameter to '$Parameter'"
+        Start-Sleep -Seconds 1
+        return $Parameter
+    }
 }
 
 function Step-One {

Original file line number	Diff line number	Diff line change
`@@ -11,9 +11,11 @@`
`11`	`11`	`param (`
`12`	`12`	`[string]$Parameter = 'Default-Parameter'`
`13`	`13`	`)`
`14`		`- Write-Verbose "Set-DefaultParameterValue: Setting Parameter to '$Parameter'"`
`15`		`- Start-Sleep -Seconds 1`
`16`		`- return $Parameter`
	`14`	`+ if ($PSCmdlet.ShouldProcess($Parameter, 'Set default parameter value')) {`
	`15`	`+ Write-Verbose "Set-DefaultParameterValue: Setting Parameter to '$Parameter'"`
	`16`	`+ Start-Sleep -Seconds 1`
	`17`	`+ return $Parameter`
	`18`	`+ }`
`17`	`19`	`}`
`18`	`20`
`19`	`21`	`function Step-One {`