@asynccontextmanager decorated functions are not callable like @contextmanager

[fried]

BPO	38415
Nosy	@ambv, @serhiy-storchaka, @1st1, @graingert, @fried
PRs	bpo-38415 @asynccontextmanager as decorators like @contextmanager #16667 bpo-38415: Remove redundant AsyncContextDecorator.__call__ override from _AsyncG… #30233

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2021-09-23.21:37:12.435>
created_at = <Date 2019-10-08.20:45:49.255>
labels = ['type-feature', 'library', '3.11']
title = '@asynccontextmanager decorated functions are not callable like @contextmanager'
updated_at = <Date 2022-02-26.12:05:50.881>
user = 'https://github.com/fried'

bugs.python.org fields:

activity = <Date 2022-02-26.12:05:50.881>
actor = 'serhiy.storchaka'
assignee = 'none'
closed = True
closed_date = <Date 2021-09-23.21:37:12.435>
closer = 'lukasz.langa'
components = ['Library (Lib)']
creation = <Date 2019-10-08.20:45:49.255>
creator = 'fried'
dependencies = []
files = []
hgrepos = []
issue_num = 38415
keywords = ['patch']
message_count = 5.0
messages = ['354232', '402528', '402529', '409056', '414094']
nosy_count = 5.0
nosy_names = ['lukasz.langa', 'serhiy.storchaka', 'yselivanov', 'graingert', 'fried']
pr_nums = ['16667', '30233']
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'enhancement'
url = 'https://bugs.python.org/issue38415'
versions = ['Python 3.11']

[fried]

The standard contextmanager decorator produces a wrapper that itself can be used as a decorator

@contextmanager
def some_context():
    ...
    yield

@some_context()
def some_function():
    # we are inside a with some_context() now. 
    ...

When I created a version of asynccontextmanager internally before it was available in the stdLib I copied this behavior and I have people internally to facebook using this behavior, Was there a reason this behavior was not replicated to asynccontextmanager?

I have a diff an tests to add

[ambv]

New changeset 86b833b by Jason Fried in branch 'main':
bpo-38415: Allow using @asynccontextmanager-made ctx managers as decorators (GH-16667)
86b833b

[ambv]

Look, not even 23 months to land this thing!

Thanks, Jason! ✨ 🍰 ✨

[graingert]

actually it was already done in 13 months!

[serhiy-storchaka]

New changeset b57dbe5 by Thomas Grainger in branch 'main':
bpo-38415: Remove redundant AsyncContextDecorator.__call__ override from _AsyncGeneratorContextManager (GH-30233)
b57dbe5