Address code review feedback: fix duplicate if-else and use & for script calls in PowerShell context

Co-authored-by: atifaziz <20511+atifaziz@users.noreply.github.com>

Author: Copilot

.github/workflows/ci.yml

@@ -85,7 +85,7 @@ jobs:
           $packParams.VersionSuffix = $versionSuffix
         }
         
-        dotnet pwsh ./build.ps1 -Pack @packParams
+        & ./build.ps1 -Pack @packParams
         if ($LASTEXITCODE) { throw "Pack failed" }
 
         Get-ChildItem -File -Filter docopt.net.*.nupkg dist |

build.ps1

@@ -115,11 +115,7 @@ try {
         }
 
         Write-Host "`n=== Running Tests ===" -ForegroundColor Green
-        if ($NoBuild) {
-            Invoke-DotNet 'test', '--no-build', '--configuration', $Configuration -AllowTestFailures
-        } else {
-            Invoke-DotNet 'test', '--no-build', '--configuration', $Configuration -AllowTestFailures
-        }
+        Invoke-DotNet 'test', '--no-build', '--configuration', $Configuration -AllowTestFailures
 
         # Note: Roslyn 4.4 analyzer is validated through integration tests
         # that use the packed NuGet package containing both analyzer variants

tests/Integration/run.ps1

@@ -24,7 +24,7 @@ try
     }
     Remove-Item -Recurse -Force (Join-Path $props.RestorePackagesPath docopt.net) -ErrorAction SilentlyContinue
     if (!$noPack) {
-        dotnet pwsh ../../build.ps1 -Pack -Configuration Release
+        & ../../build.ps1 -Pack -Configuration Release
         if ($LASTEXITCODE) { throw }
     }
     Remove-Item bin, obj -Recurse -Force -ErrorAction SilentlyContinue