Skip to content
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.

Sign up for GitHub

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

Jump to bottom

asyncio: Document return values of AbstractEventLoop.remove_{reader,writer} #99830

Closed
bdarnell opened this issue Nov 28, 2022 · 3 comments · Fixed by #100302
Closed

asyncio: Document return values of AbstractEventLoop.remove_{reader,writer} #99830

bdarnell opened this issue Nov 28, 2022 · 3 comments · Fixed by #100302
Labels
docs Documentation in the Doc dir stdlib Python modules in the Lib dir topic-asyncio

Comments

@bdarnell
Copy link
Contributor

bdarnell commented Nov 28, 2022 •
edited by bedevere-bot
Loading

Documentation

According to python/typeshed#7042, AbstractEventLoop.remove_{reader,writer} return a bool, but there is no indication of this in the documentation for these methods. These return values need to be documented so that subclass implementors can know how to implement them appropriately.

Linked PRs
@bdarnell bdarnell added the docs Documentation in the Doc dir label Nov 28, 2022
@hugovk hugovk added topic-asyncio stdlib Python modules in the Lib dir labels Nov 28, 2022
@ezio-melotti ezio-melotti moved this to Todo in asyncio Nov 28, 2022
@kumaraditya303
Copy link
Contributor

Would you like to create a PR for this?

bdarnell added a commit to bdarnell/cpython that referenced this issue Dec 16, 2022
@bdarnell
pythongh-99830: asyncio: Document returns of remove_{reader,writer}
8842f42 
Fixes python#99830
@bdarnell bdarnell mentioned this issue Dec 16, 2022
Merged
@bdarnell
Copy link
Contributor Author

Sure: #100302

Repository owner moved this from Todo to Done in asyncio Dec 16, 2022
gvanrossum pushed a commit that referenced this issue Dec 16, 2022
@bdarnell
gh-99830: asyncio: Document returns of remove_{reader,writer} (#100302)
5234e1c
@bedevere-bot bedevere-bot mentioned this issue Dec 16, 2022
Merged
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Dec 16, 2022
@bdarnell @miss-islington
pythongh-99830: asyncio: Document returns of remove_{reader,writer} (p…
9a25bb9 
…ythonGH-100302)

(cherry picked from commit 5234e1c)

Co-authored-by: Ben Darnell <[email protected]>
@bedevere-bot bedevere-bot mentioned this issue Dec 16, 2022
Merged
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Dec 16, 2022
@bdarnell @miss-islington
pythongh-99830: asyncio: Document returns of remove_{reader,writer} (p…
fb54e42 
…ythonGH-100302)

(cherry picked from commit 5234e1c)

Co-authored-by: Ben Darnell <[email protected]>
@gvanrossum
Copy link
Member

Thanks, Ben!

miss-islington added a commit that referenced this issue Dec 16, 2022
@miss-islington @bdarnell
gh-99830: asyncio: Document returns of remove_{reader,writer} (GH-100302
4c810f9 
)

(cherry picked from commit 5234e1c)

Co-authored-by: Ben Darnell <[email protected]>
miss-islington added a commit that referenced this issue Dec 16, 2022
@miss-islington @bdarnell
gh-99830: asyncio: Document returns of remove_{reader,writer} (GH-100302
958be79 
)

(cherry picked from commit 5234e1c)

Co-authored-by: Ben Darnell <[email protected]>
carljm added a commit to carljm/cpython that referenced this issue Dec 16, 2022
@carljm
Merge branch 'main' into fastmock
1b6241a 
* main:
  pythongh-99540: Constant hash for _PyNone_Type to aid reproducibility (pythonGH-99541)
  pythongh-100039: enhance __signature__ to work with str and callables (pythonGH-100168)
  pythongh-99830: asyncio: Document returns of remove_{reader,writer} (python#100302)
  "Compound statement" docs: Fix with-statement step indexing (python#100286)
  pythonGH-90043: Handle NaNs in COMPARE_OP_FLOAT_JUMP (pythonGH-100278)
shihai1991 added a commit to shihai1991/cpython that referenced this issue Dec 18, 2022
@shihai1991
Merge remote-tracking branch 'origin/main' into bpo_34697
2878571 
* origin/main: (1306 commits)
  Correct CVE-2020-10735 documentation (python#100306)
  pythongh-100272: Fix JSON serialization of OrderedDict (pythonGH-100273)
  pythongh-93649: Split tracemalloc tests from _testcapimodule.c (python#99551)
  Docs: Use `PY_VERSION_HEX` for version comparison (python#100179)
  pythongh-97909: Fix markup for `PyMethodDef` members (python#100089)
  pythongh-99240: Reset pointer to NULL when the pointed memory is freed in argument parsing (python#99890)
  pythongh-99240: Reset pointer to NULL when the pointed memory is freed in argument parsing (python#99890)
  pythonGH-98831: Add DECREF_INPUTS(), expanding to DECREF() each stack input (python#100205)
  pythongh-78707: deprecate passing >1 argument to `PurePath.[is_]relative_to()` (pythonGH-94469)
  pythongh-99540: Constant hash for _PyNone_Type to aid reproducibility (pythonGH-99541)
  pythongh-100039: enhance __signature__ to work with str and callables (pythonGH-100168)
  pythongh-99830: asyncio: Document returns of remove_{reader,writer} (python#100302)
  "Compound statement" docs: Fix with-statement step indexing (python#100286)
  pythonGH-90043: Handle NaNs in COMPARE_OP_FLOAT_JUMP (pythonGH-100278)
  Improve stats presentation for calls. (pythonGH-100274)
  Better stats for `LOAD_ATTR` and `STORE_ATTR` (pythonGH-100295)
  pythongh-81057: Move the Cached Parser Dummy Name to _PyRuntimeState (python#100277)
  Document that zipfile's pwd parameter is a `bytes` object (python#100209)
  pythongh-99767: mark `PyTypeObject.tp_watched` as internal use only in table (python#100271)
  Fix typo in introduction.rst (python#100266)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
docs Documentation in the Doc dir stdlib Python modules in the Lib dir topic-asyncio
Projects
asyncio
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

gh-99830: asyncio: Document returns of remove_{reader,writer} bdarnell/cpython
4 participants
@bdarnell @hugovk @gvanrossum @kumaraditya303