Skip to content

Add DSC-related compound words to spell-check expect list#45195

Closed
Copilot wants to merge 2 commits intodev/vanzue/pt-ext-localizationfrom
copilot/sub-pr-45194
Closed

Add DSC-related compound words to spell-check expect list#45195
Copilot wants to merge 2 commits intodev/vanzue/pt-ext-localizationfrom
copilot/sub-pr-45194

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 29, 2026

Summary of the Pull Request

Added missing DSC-related compound words to the spell-check expect list to resolve unrecognized word warnings in DscResources.wxs.

PR Checklist

  • Communication: I've discussed this with core contributors already. If the work hasn't been agreed, this work might be rejected
  • Tests: Added/updated and all pass
  • Localization: All end-user-facing strings can be localized
  • Dev docs: Added/updated
  • New binaries: Added on the required places
  • Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Updated .github/actions/spell-check/expect.txt with three compound words used in the DSC installer configuration:

  • DscJsonFiles
  • DscJsonFilesPath
  • DscResourcesComponentGroup

These identifiers are referenced in installer/PowerToysSetupVNext/DscResources.wxs and were triggering spell-check warnings. Entries added in alphabetical order.

Validation Steps Performed

  • Verified alphabetical ordering in expect.txt maintained
  • Confirmed words match usage in DscResources.wxs

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: vanzue <69313318+vanzue@users.noreply.github.com>
Copilot AI changed the title [WIP] Add localization files to PowerToys installer Add DSC-related compound words to spell-check expect list Jan 29, 2026
Copilot AI requested a review from vanzue January 29, 2026 14:20
@vanzue vanzue closed this Jan 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants