cspell issues in cpp branch , needs updated node, and cspell and npx …

…need to be installed for cspell to run (Azure#32255) Co-authored-by: George Arama <50641385+gearama@users.noreply.github.com>
azure-sdk · Nov 2, 2022 · 5d0988c · 5d0988c
1 parent 567a36d
commit 5d0988c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/eng/common/pipelines/templates/steps/check-spelling.yml b/eng/common/pipelines/templates/steps/check-spelling.yml
@@ -18,8 +18,8 @@ steps:
     - task: NodeTool@0
       condition: and(succeededOrFailed(), ne(variables['Skip.SpellCheck'],'true'))
       inputs:
-        versionSpec: 16.x
-      displayName: Use Node.js 16.x
+        versionSpec: 18.x
+      displayName: Use Node.js 18.x
 
     - task: PowerShell@2
       displayName: Check spelling (cspell)

diff --git a/eng/common/spelling/Invoke-Cspell.ps1 b/eng/common/spelling/Invoke-Cspell.ps1
@@ -167,10 +167,12 @@ $originalLocation = Get-Location
 
 try {
   Set-Location $PackageInstallCache
+  npm install npx | Out-Null
+  npm install cspell | Out-Null
   npm install | Out-Null
 
   # Use the mutated configuration file when calling cspell
-  $command = "npx --no-install cspell $JobType --config $CSpellConfigPath --no-must-find-files --root $SpellCheckRoot --relative"
+  $command = "npx cspell $JobType --config $CSpellConfigPath --no-must-find-files --root $SpellCheckRoot --relative"
   Write-Host $command
   $cspellOutput = npx  `
     --no-install `