Skip to content

gh-91323: Revert "Allow overriding a future compliance check in asyncio.Task (GH-32197)"#95442

Merged
ambv merged 2 commits into
python:mainfrom
ambv:revert-gh-32197
Aug 4, 2022
Merged

gh-91323: Revert "Allow overriding a future compliance check in asyncio.Task (GH-32197)"#95442
ambv merged 2 commits into
python:mainfrom
ambv:revert-gh-32197

Conversation

@ambv
Copy link
Copy Markdown
Contributor

@ambv ambv commented Jul 29, 2022

@ambv ambv added the skip news label Jul 29, 2022
@ambv
Copy link
Copy Markdown
Contributor Author

ambv commented Jul 29, 2022

I marked this as "skip news" as the original change wasn't mentioned in the change log.

@ambv ambv added the needs backport to 3.11 only security fixes label Jul 29, 2022
@ambv ambv changed the title Revert "bpo-47167: Allow overriding a future compliance check in asyncio.Task (GH-32197)" bpo-47167: Revert "Allow overriding a future compliance check in asyncio.Task (GH-32197)" Jul 29, 2022
@ambv ambv changed the title bpo-47167: Revert "Allow overriding a future compliance check in asyncio.Task (GH-32197)" gh-91323: Revert "Allow overriding a future compliance check in asyncio.Task (GH-32197)" Jul 29, 2022
Copy link
Copy Markdown
Member

@1st1 1st1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, according to #91323 (comment) this change isn't really needed.

I don't think it's much useful for anything else and I think the bar for adding new APIs to Tasks/Future is very high. Let's drop it for now. cc @asvetlov

@miss-islington
Copy link
Copy Markdown
Contributor

Thanks @ambv for the PR 🌮🎉.. I'm working now to backport this PR to: 3.11.
🐍🍒⛏🤖

@ambv ambv deleted the revert-gh-32197 branch August 4, 2022 13:51
@bedevere-bot
Copy link
Copy Markdown

GH-95652 is a backport of this pull request to the 3.11 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.11 only security fixes label Aug 4, 2022
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Aug 4, 2022
… asyncio.Task (pythonGH-32197)" (pythonGH-95442)

This reverts commit d4bb38f.
(cherry picked from commit 0342c93)

Co-authored-by: Łukasz Langa <lukasz@langa.pl>
ambv added a commit that referenced this pull request Aug 4, 2022
…io.Task (GH-32197)" (GH-95442) (GH-95652)

This reverts commit d4bb38f.
(cherry picked from commit 0342c93)

Co-authored-by: Łukasz Langa <lukasz@langa.pl>
@ambv ambv restored the revert-gh-32197 branch August 4, 2022 16:41
@ambv ambv deleted the revert-gh-32197 branch August 5, 2022 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants