Skip to content

Conversation

@azure-sdk
Copy link
Collaborator

Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#12569 See eng/common workflow

@azure-sdk azure-sdk requested a review from a team as a code owner October 27, 2025 18:48
@azure-sdk azure-sdk added EngSys This issue is impacting the engineering system. Central-EngSys This issue is owned by the Engineering System team. labels Oct 27, 2025
@azure-sdk azure-sdk added EngSys This issue is impacting the engineering system. Central-EngSys This issue is owned by the Engineering System team. labels Oct 27, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR syncs the eng/common directory with azure-sdk-tools to update the spell-checking infrastructure. The changes upgrade cspell from version 6.12.0 to 9.2.1 and refactor how files are passed to the spell checker.

Key changes:

  • Upgrades cspell dependency from ^6.12.0 to ^9.2.1 and removes bundled dictionaries
  • Refactors Invoke-Cspell.ps1 to accept files via stdin using pipeline input instead of glob patterns
  • Updates parameter name from ScanGlobs to FileList in check-spelling-in-changed-files.ps1

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.

File Description
eng/common/spelling/package.json Bumps version to 1.0.0 and upgrades cspell to 9.2.1
eng/common/spelling/Invoke-Cspell.ps1 Refactors to use stdin for file input with begin/process/end blocks
eng/common/scripts/check-spelling-in-changed-files.ps1 Updates parameter name from ScanGlobs to FileList

@azure-sdk azure-sdk force-pushed the sync-eng/common-djurek/cspell-upgrade-12569 branch from dea458c to eccbff1 Compare October 29, 2025 07:05
@azure-sdk azure-sdk force-pushed the sync-eng/common-djurek/cspell-upgrade-12569 branch from eccbff1 to 8512598 Compare October 29, 2025 21:41
@azure-sdk azure-sdk merged commit 190d4f1 into main Oct 30, 2025
16 checks passed
@azure-sdk azure-sdk deleted the sync-eng/common-djurek/cspell-upgrade-12569 branch October 30, 2025 17:43
JennyPng pushed a commit to JennyPng/azure-sdk-for-python that referenced this pull request Nov 3, 2025
)

* Upgrade cspell

* Use stdin for cspell input

* Add pipeline support to Invoke-Cspell.ps1

* Quote parameters

* Only quote the needed

---------

Co-authored-by: Daniel Jurek <djurek@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Central-EngSys This issue is owned by the Engineering System team. EngSys This issue is impacting the engineering system.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants