Uh oh!
There was an error while loading. Please reload this page.
Fix typos in >60 files using codespell & manual review to get rid of false positives - #136887
Fix typos in >60 files using codespell & manual review to get rid of false positives#136887corneliusroemer wants to merge 9 commits into
Conversation
AA-Turner
commented
Jul 20, 2025
Please avoid large PRs like this, they're hard to review and generally unhelpful (e.g. spelling errors in comments have no real impact). A |
ZeroIntensity
commented
Jul 20, 2025
Yeah, I feel like I inspired this by approving #136884. We want typo fixes in public documentation and docstrings, we don't care too much about comments. |
I checked discourse and comment fixes were in-scope per @brettcannon: https://discuss.python.org/t/fixing-simple-typo-in-comment-not-document/2297/2 So you prefer more but smaller PRs focused on docstrings/public documentation? |
AA-Turner
commented
Jul 20, 2025
I'd prefer that we spent resources working on more productive things than spelling mistakes! If you do choose to submit these changes again, though, please have only have one PR open at once and do the changes in small batches, where end users are most affected. A |
ZeroIntensity
commented
Jul 20, 2025
To be fair, that post is from 2019. I think our workflow has changed a bit since then. |
If you'd like me to get rid of non-super-unanimous fixes and/or split into multiple PRs let me know.
Examples of "arguable" fixes / common "errors":
Codespell is overly opinionated in many cases, but I've gotten rid of these already (e.g. it really doesn't like re-use, this would cause ~50 more changes which I've rejected already)
📚 Documentation preview 📚: https://cpython-previews--136887.org.readthedocs.build/