Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit bf3fdb6

Browse files
committed
Remove logging
1 parent 5091ff3 commit bf3fdb6

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

vscode-powershell.build.ps1

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,6 @@ $SCRIPT:ErrorActionPreference = 'Stop'
2323
#region Prerequisites
2424

2525
task Prerequisites {
26-
[CmdletBinding()] #This is just so we have access to $PSCmdlet
27-
param()
2826
# We want all prereqs to run so we can tell the user everything they are missing, rather than them having to fix/check/fix/check/etc.
2927
$requirements = Import-PowerShellDataFile -Path $RequirementsManifest
3028
$ErrorActionPreference = 'Continue'
@@ -36,7 +34,7 @@ task Prerequisites {
3634

3735
if ($preRequisiteIssues) {
3836
Write-Warning 'The following prerequisites are not met. Please install them before continuing. Setup guidance can be found here: https://github.com/PowerShell/vscode-powershell/blob/main/docs/development.md'
39-
$preRequisiteIssues | ForEach-Object { $PSCmdlet.WriteError($_) }
37+
$preRequisiteIssues
4038
exit 1
4139
}
4240
# We dont need to test for vscode anymore because we download it dynamically during the build.
@@ -46,10 +44,6 @@ task Prerequisites {
4644
function Assert-Pwsh ([Version]$RequiredPowerShellVersion) {
4745
$ErrorActionPreference = 'Continue'
4846
try {
49-
if ($isLinux) {
50-
Write-Host -Fore Magenta "pwsh which: $(which pwsh)"
51-
}
52-
Write-Host -Fore Magenta "pwsh command: $(Get-Command pwsh)"
5347
[Version]$pwshVersion = (& pwsh -v) -replace '^PowerShell '
5448
} catch {
5549
if ($InstallPrerequisites) {
@@ -81,7 +75,12 @@ function Assert-NodeVersion ([Version]$RequiredNodeVersion) {
8175
Write-Debug "PREREQUISITE: Detected supported Node.js version $nodeVersion at or above minimum $RequiredNodeVersion"
8276
}
8377

84-
function Assert-Module ([ModuleSpecification[]]$RequiredModules) {
78+
function Assert-Module {
79+
[CmdletBinding()]
80+
param(
81+
[ModuleSpecification[]]$RequiredModules
82+
)
83+
8584
$ErrorActionPreference = 'Continue'
8685
foreach ($moduleSpec in $RequiredModules) {
8786
Write-Debug "PREREQUISITE: Checking for module $($moduleSpec.Name) $($moduleSpec.RequiredVersion)"
@@ -91,6 +90,7 @@ function Assert-Module ([ModuleSpecification[]]$RequiredModules) {
9190

9291
if (-not $moduleMatch) {
9392
if ($InstallPrerequisites) {
93+
#We want to install the module in both Windows PowerShell and PowerShell for testing.
9494
if ($isWindows -or $PSVersionTable.PSEdition -eq 'Desktop') {
9595
$otherPowershell = if ($PSVersionTable.PSVersion -lt '6.0.0') {
9696
'pwsh'

0 commit comments

Comments
 (0)