From 4ba48a3ef868bd34b5424041e56b2d187f34a3a1 Mon Sep 17 00:00:00 2001 From: Azure SDK Bot <53356347+azure-sdk@users.noreply.github.com> Date: Fri, 3 Mar 2023 07:28:59 -0800 Subject: [PATCH] Sync eng/common directory with azure-sdk-tools for PR 5595 (#20322) * Use "npm ci" to install cspell and respect package-lock.json * Review feedback * Pipe npm ci output to Write-Host --------- Co-authored-by: Daniel Jurek --- eng/common/spelling/Invoke-Cspell.ps1 | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/eng/common/spelling/Invoke-Cspell.ps1 b/eng/common/spelling/Invoke-Cspell.ps1 index 79205ed90643..9fc5dec9b991 100644 --- a/eng/common/spelling/Invoke-Cspell.ps1 +++ b/eng/common/spelling/Invoke-Cspell.ps1 @@ -25,7 +25,7 @@ created in the temp folder, package*.json files will be placed in that folder. .PARAMETER LeavePackageInstallCache If set the PackageInstallCache will not be deleted. Use if there are multiple calls to Invoke-Cspell.ps1 to prevent creating multiple working directories and -redundant calls `npm install`. +redundant calls `npm ci`. .PARAMETER Test Run test functions against the script logic @@ -167,9 +167,7 @@ $originalLocation = Get-Location try { Set-Location $PackageInstallCache - npm install npx | Out-Null - npm install cspell | Out-Null - npm install | Out-Null + npm ci | Write-Host # Use the mutated configuration file when calling cspell $command = "npx cspell $JobType --config $CSpellConfigPath --no-must-find-files --root $SpellCheckRoot --relative"