Uh oh!
There was an error while loading. Please reload this page.
Change __future__ docs description to avoid confusion with concurrency futures - #92760
Change __future__ docs description to avoid confusion with concurrency futures#92760arhadthedev wants to merge 8 commits into
__future__ docs description to avoid confusion with concurrency futures#92760Conversation
carljm
left a comment
There was a problem hiding this comment.
Looks like an improvement to me.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Co-authored-by: Carl Meyer <carl@oddbird.net>
arhadthedev
commented
Jul 18, 2022
JelleZijlstra
commented
Jul 23, 2022
I don't find the current wording unclear, and the proposed new wording is so long that it will make the page title spill into multiple lines on many screens. If another core dev has a different opinion, I won't object to merging this though. |
@JelleZijlstra Will Edit: for comparison: |
arhadthedev
commented
Jul 30, 2022
Applied the shorter wording described above. |
__future__ docs description from concurrency futures__future__ docs description to avoid confusion with concurrency futuresarhadthedev
commented
Oct 6, 2022
Ping. |
JelleZijlstra
commented
Oct 6, 2022
Sorry, I still don't see a reason for change here. I'm leaving this open in case another core dev thinks there's room for improvement, but I'm not planning to merge anything. |
Currently, the library index page describes
__future__as:However, it creates confusion with futures from the concurrency domain. So this PR changes the description to:
Edit: replaced with a shorter wording: