⚙️ [Maintenance]: Add afterall to codespell ignore words list#553
Conversation
No Significant Changes DetectedThis PR does not contain changes to files that would trigger a new release:
Build, test, and publish stages will be skipped for this PR. If you believe this is incorrect, please verify that your changes are in the correct locations. |
No Significant Changes DetectedThis PR does not contain changes to files that would trigger a new release:
Build, test, and publish stages will be skipped for this PR. If you believe this is incorrect, please verify that your changes are in the correct locations. |
Marius Storhaug (MariusStorhaug)
marked this pull request as ready for review
Marius Storhaug (MariusStorhaug)
requested a review
from a team
as a code owner
There was a problem hiding this comment.
Pull request overview
This PR addresses false positive spell check warnings by adding afterall to the codespell ignore words list. The term AfterAll is a legitimate Pester 5.x keyword used extensively in the test suite for cleanup blocks after test contexts. Since codespell performs case-insensitive checks, it was incorrectly flagging this as a misspelling of "after all" (two words).
Changes:
- Added
ignore-words-list = afterallto.codespellrcto prevent codespell from flagging Pester'sAfterAllkeyword as a misspelling
Marius Storhaug (MariusStorhaug)
merged commit 39b98ce
into
main
Marius Storhaug (MariusStorhaug)
deleted the
maintenance/codespellrc-ignore-afterall
branch
1 participant
Adds
afterallto the codespell ignore words list in.codespellrcto prevent false positive spell check warnings in CI linting. This term is commonly used in PowerShell (e.g., Pester'sAfterAllblock) and should not be flagged as a misspelling.Codespell configuration
The
.codespellrclinter configuration now includesignore-words-list = afterall, which tells codespell to skip this word during spell checking. No changes to functionality or shipped artifacts.