Skip to content

PEP 554 (text) updates #20

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

Open
3 tasks
ericsnowcurrently opened this issue Aug 24, 2018 · 3 comments
Open
3 tasks

PEP 554 (text) updates #20

ericsnowcurrently opened this issue Aug 24, 2018 · 3 comments
Assignees
Labels
complexity: low easy to figure out P0 high priority size: small requires few changes type: community URGENT must be dealt with ASAP

Comments

@ericsnowcurrently
Copy link
Owner

A few things have changed since the last time the PEP was updated:

  • ...
  • ...
  • ...
@ericsnowcurrently ericsnowcurrently added P0 high priority URGENT must be dealt with ASAP size: small requires few changes complexity: low easy to figure out type: community labels Aug 24, 2018
@ericsnowcurrently ericsnowcurrently self-assigned this May 7, 2019
@dimitern
Copy link

dimitern commented Nov 8, 2019

A minor typo I found here: https://www.python.org/dev/peps/pep-0554/#passing-objects-via-marshal
Not sure a comment is the proper way to report this - it's minor anyway.

interp = interpreters.create()
r, s = interpreters.create_fifo()

should most likely be:

interp = interpreters.create()
r, s = interpreters.create_channel()

Happy to file an issue if would be helpful, or even PR if appropriate.

@ericsnowcurrently
Copy link
Owner Author

Hey @dimitern! Yeah, an issue and PR would be great! Both should be made against the https://github.com/python/peps repo.

@ericsnowcurrently
Copy link
Owner Author

Please @-mention me on the issue you create.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
complexity: low easy to figure out P0 high priority size: small requires few changes type: community URGENT must be dealt with ASAP
Projects
None yet
Development

No branches or pull requests

2 participants