-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PEP 734: Multiple Interpreters in the Stdlib #3523
PEP 734: Multiple Interpreters in the Stdlib #3523
Conversation
1a8c929
to
4d3f06e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi Eric, I know we already spoke about some of this off-line, but I ran out of time reviewing all my comments. I know the next revision will be much improved!
@gvanrossum, I've made updates to address your review comments. Thanks again for taking the time to provide such useful feedback. ❤️ I'm concerned that the PEP is getting a little too long, so let me know if there are any sections that you think are unnecessary or have too much detail, if you can spare me the time. |
FYI, once this PR is merged, I plan on updating the implementation (and update some benchmarks, to try it out) before posting the PEP for discussion. That way I can tweak the PEP if something doesn't feel right in practice. |
Are you awaiting more review from me? Im in Japan but can make a little time. |
Sweet of you to offer, but no. If needed, we can chat when you are back. |
There are probably a handful of things to tweak, like trim back some of the fluff, but I'll do that in follow-up PRs. |
Basic requirements (all PEP Types)
pep-NNNN.rst
), PR title (PEP 123: <Title of PEP>
) andPEP
headerAuthor
orSponsor
, and formally confirmed their approvalAuthor
,Status
(Draft
),Type
andCreated
headers filled out correctlyPEP-Delegate
,Topic
,Requires
andReplaces
headers completed if appropriate.github/CODEOWNERS
for the PEPStandards Track requirements
Python-Version
set to valid (pre-beta) future Python version, if relevantDiscussions-To
andPost-History
📚 Documentation preview 📚: https://pep-previews--3523.org.readthedocs.build/pep-0734/