xref #2208 and #2209 for the implementation update PRs.

A

Looks fine to me, though now there's a merge conflict. Would you be able to address the conflict?

I've got to get on a flight now, but I'll try to look over it late tonight or early tomorrow; sorry for the long delay—had a complex trip to visit various family members, with very limited time and internet over the past couple weeks. So far, it looks very good though.

Looks like the merge conflict is due to #2211 which fixes a bunch of trivial typos in a smattering of PEPs...which is already fixed in this PR (as well as #2200 ) and thus conflicts with both, and possibly others...which the automated implementation in #2151 would help address, but I digress.

I couldn't work out how to keep @CAM-Gerlach as the 'author' of the formatting commit whilst re-writing history, so I added him as co-author on the commits with the Co-authored-by: syntax.

Unacceptable, I say. Unacceptable! 😤 /s

Thanks for thinking of that, and no worries at all—it was as much your work as mine! As a note for the future, for situations where this is important, you can either use the GIT_* env vars or, as a hack, steal one of my original commits, apply the patch to the working tree and then --amend it it into that commit—it will list you as commiter (as it should), but I'll still be listed as the author. But in this case, that's way too much work for something that doesn't really matter—in fact, I think it makes more sense for you to be the author anyway, since you did manually author the changes.

@Mariatta conflict resolved -- thanks for pointing it out

A

@CAM-Gerlach quick ping :) I don't know if you've had any time to review?

use the GIT_* env vars

I had no idea these existed! Learn a new thing every day...

A

Sorry, busy with holiday travel and visiting family, but from what I skimmed it looked fine to me, thanks.