Skip to content

Commit

Permalink
Sync eng/common directory with azure-sdk-tools for PR 5595 (#34669)
Browse files Browse the repository at this point in the history
* 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 <djurek@microsoft.com>
  • Loading branch information
azure-sdk and danieljurek authored Mar 3, 2023
1 parent e3f5a72 commit 605af21
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions eng/common/spelling/Invoke-Cspell.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down Expand Up @@ -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"
Expand Down

0 comments on commit 605af21

Please sign in to comment.