Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 26, 2025

Summary of the Pull Request

Corrected British English "cancelled" to American English "canceled" in PowerDisplay module comments to align with PowerToys codebase standards. Comprehensive spell-check performed using codespell to ensure no other spelling issues exist.

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

Changed 4 instances across 3 files:

  • DisplayChangeWatcher.cs: OperationCanceledException comment
  • SimpleDebouncer.cs: Two comments in debounce logic
  • PowerDisplayPage.xaml.cs: ComboBox revert comment

Comment-only changes. No functional impact.

Validation Steps Performed

  • ✅ Verified all PowerDisplay-related files for spelling consistency with codebase standards
  • ✅ Ran codespell on all PowerDisplay module source files
  • ✅ Ran codespell on Settings UI PowerDisplay pages and ViewModels
  • ✅ Ran codespell on PowerDisplay documentation
  • ✅ Confirmed no additional spelling errors exist in the PowerDisplay module

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

Copilot AI changed the title [WIP] Introduce new utility PowerDisplay to manage monitor settings Fix spelling: standardize to American English 'canceled' Dec 26, 2025
Copilot AI requested a review from moooyo December 26, 2025 09:43
@moooyo
Copy link
Contributor

moooyo commented Dec 26, 2025

@copilot Please fix all spelling issue.

Copy link
Contributor Author

Copilot AI commented Dec 26, 2025

@copilot Please fix all spelling issue.

All spelling issues in the PowerDisplay module have been fixed. I ran codespell on all PowerDisplay-related files including:

  • src/modules/powerdisplay/
  • Settings UI PowerDisplay files
  • Documentation in doc/devdocs/modules/powerdisplay/

The only spelling issues found were the 4 instances of "cancelled" → "canceled" that were already fixed in commit 55044ff. No additional spelling errors were detected.

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