Thanks for the cleanup. I can review it in more detail after a few housekeeping updates have been made (updating the branch to current HEAD, creating an issue, and adding an issue number).

You can just delete the page feed line (see PEP 8, two blank lines only between top level constructs). The page feeds are a historical holdover for when this library was developed independently.

To appease Bedevere about the issue number, please prefix the PR title as "gh-XXXXX:"

Hi @warsaw, thanks for the quick response. I've added an issue, rebased to the current HEAD and removed that extra line as you requested. Should be good for a review now.

@hugovk cheers for the headsup r.e the issue number.

If you used a program/linter to find these, please add 'found by xxxx' to the first sentence in the message.

Also an unused msg here (found by Flake8):

There are 45 pagebreaks remaining in the entire cpython repo, 34 of which are in email. I'm happy to replace them with newlines / delete them as appropriate per pep-8 if people want them gone. Might be something for another issue though:)

There are 45 pagebreaks remaining in the entire cpython repo, 34 of which are in email. I'm happy to replace them with newlines / delete them as appropriate per pep-8 if people want them gone. Might be something for another issue though:)

I agree with @warsaw that it's now time for these to go, but it would be better to remove all of them from the stdlib at once, in a separate PR, I think, rather than in this PR.

(Not requesting a change to this PR; it's fine to remove the one you're already removing, in my opinion, since you're already touching that file :)

Inspired by @AlexWaygood 's use of the tool to find issues in the test_typing.py file (can't seem to find the exact issue).

This is the one:

• Several minor bugs in test_typing highlighted by pyflakes #102444

I appear to have duplicated the commits during the merge somehow. I would rebase and drop them but I'm under the impression that force-pushing is discouraged. If we merge this PR they'll all be squashed anyways as I understand it.. Let me know if dropping the duplicated commits is preferable and I can do that:)

Correct, all PRs are squashed and merged, so don't worry about the individual commits too much. 👍

Hi @warsaw. Could you please take a look at this when you get a chance?