Typos found by codespell by DimitriPapadopoulos · Pull Request #2073 · python/peps · GitHub
Skip to content

Typos found by codespell - #2073

Merged
brettcannon merged 2 commits into
python:masterfrom
DimitriPapadopoulos:codespell
Sep 17, 2021
Merged

brettcannon merged 2 commits into
python:masterfrom
DimitriPapadopoulos:codespell

Conversation

@DimitriPapadopoulos

Copy link
Copy Markdown
Contributor

These typos detected by codespell are scattered over many PEPs. Should I really open a different PR for each PEP?

Comment thread pep-0400.txt Outdated
Comment thread pep-0454.txt
Comment thread pep-0659.rst Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"warmup" seems fine

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

True, but codespell prefers warm-up and so do Merriam-Webster and The Oxford Advanced Learner's Dictionary.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd prefer to leave the original author's wording unless it's clearly wrong. English spelling has some variation.

Similarly, you changed "reenable" to "re-enable" at some point. That can also stay as "reenable".

@DimitriPapadopoulos DimitriPapadopoulos Sep 17, 2021

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, I'll keep it as it was.

As for re-enable, it's probably my wrong. I seem to recall it is a citation of a mail and shouldn't be changed. I'll look into it and fix it.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That said, PEP 0659 itself is inconsistent:

"warm up" time
warmup times

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

@gvanrossum

Copy link
Copy Markdown
Member

Honestly I don’t like such “omnibus” PRs. There’s often something that eluded review.

And I don’t mind having the occasional typo, unless it makes the text hard to follow. The time to use a spell checker is when first writing the text.

@DimitriPapadopoulos

DimitriPapadopoulos commented Sep 17, 2021

Copy link
Copy Markdown
Contributor Author

Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants