PEP 639: Update the metadata version to 2.3 as 2.2 already exists #2140
Conversation
There was a problem hiding this comment.
Thanks for getting started on this! Its 100% my fault for not editing my comment (and for not doing it in the first place), but per @pfmoore 's response, that is actually the one part that I was mistaken about, and instead all the references to the metadata version should be removed instead and the PEP reformatted to look like PEP 643. I'll mention that in a review and drop a PR shortly 🤞 doing that instead. But thanks again for getting things moving!
|
Quick update: My PR is almost ready, at least 3/4ths done as of tonight, and I hope to finish and submit sometime tomorrow. To note, aside from the fact that @pfmoore requested a different approach from that here (as I've implemented), it will make for major merge conflicts with my work, so let's make sure not to merge this one, okay? :) My work is on a branch on my fork, in case anyone wants to take an early sneak peek. Side note: Given pretty much all the significant content changes were already discussed/agreed on the Discuss thread, but on the other hand it is a pretty comprehensive overhaul, should I submit it as a PR once its ready and invite the Discuss thread followers to review it, or link my branch in the Discuss thread for feedback before submitting here? |
|
@CAM-Gerlach I'd open a PR 👍 |
|
Will do, thanks, sounds like that's what Paul suggested on the Discuss thread a while back too. Got stuck with a big GitHub backlog at the moment but will try to finish by later tonight. |
|
Opened as #2164 ! |
No description provided.