RFE: Check and enforce proper spelling in CI
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ironic |
Fix Released
|
Medium
|
Adam McArthur | ||
ironic-python-agent |
Fix Released
|
Medium
|
Jay Faulkner |
Bug Description
Recently, I become aware of codespell through Nova's adoption of the tool in their codebase: https:/
Rather than using a dictionary of properly spelled words, it uses a list of known-misspellings, limiting the false positive rates significantly.
It's my belief that adding support for this tool to our tox configuration and, eventually, adding it as a lint requirement on landing changes, would lead to an overall higher quality codebase for Ironic projects.
I will push an example change to IPA showing what this would loook like. It is based on the Nova implementation of codespell as seen here: https:/
At a minimum, the resulting changeset of adding a codespell target to tox and fixing misspellings caught by it, would be an improvement even if not added as required CI for landing changes.
Changed in ironic: | |
importance: | Undecided → Medium |
Changed in ironic: | |
status: | Triaged → Fix Released |
assignee: | nobody → Adam McArthur (adamcarthur) |
To be clear: if this gets community approval/consensus; I'd extend it to all our active repositories.