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

Preview style: async with context managers aren't wrapped in parentheses #3591

Closed
yilei opened this issue Mar 1, 2023 · 0 comments · Fixed by #3609
Closed

Preview style: async with context managers aren't wrapped in parentheses #3591

yilei opened this issue Mar 1, 2023 · 0 comments · Fixed by #3609
Labels
T: style What do we want Blackened code to look like?

Comments

@yilei
Copy link
Contributor

yilei commented Mar 1, 2023

#3489 wraps multiple context managers in parentheses when targeting Python 3.9+, but async with statements aren't handled.

We noticed this in the review of #3589, but the fix is different enough that I'm filing a separate issue and will send a follow-up PR.

Example: https://black.vercel.app/?version=stable&state=_Td6WFoAAATm1rRGAgAhARYAAAB0L-Wj4AIWAK9dAD2IimZxl1N_WmufDs0LojAMwSd5y6tVHFECi48em1NmrZSWd-aOQDsyxUoj0wG4N0KjynelfrUIzSWL64h5VoX_mCOwgMPULx38--l2ByPDULdy3kY55NpHvpodxXDskAGNjzDUowTiDYEk1Nqn0zKozBat4Jr52JOK7_c0zbTbJlm2bGELr0genqmpOYe0Veq-1N1caP91OmdWsqLjL9f2ZG7iyYFCy51v8cbzDGQAACCNqwSsCCbzAAHLAZcEAADbYoSiscRn-wIAAAAABFla
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
T: style What do we want Blackened code to look like?
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Consistently format async statements similar to their non-async version. yilei/black
1 participant
@yilei