Formatter: wrap_multiple_context_managers_in_parens preview style
#8889
Assignees
Milestone
Comments
wrap_multiple_context_managers_in_parens: Requires Py39+wrap_multiple_context_managers_in_parens preview style
I would prefer to stick to our existing configuration semantics, and not introduce any autodetection. |
|
It looks like the original intent, per the docs, is to reformat as: # Input
with make_context_manager1() as cm1, make_context_manager2() as cm2, make_context_manager3() as cm3, make_context_manager4() as cm4:
pass
# < Python 3.9
with \
make_context_manager1() as cm1, \
make_context_manager2() as cm2, \
make_context_manager3() as cm3, \
make_context_manager4() as cm4 \
:
pass
# >= Python 3.9
with (
make_context_manager1() as cm1,
make_context_manager2() as cm2,
make_context_manager3() as cm3,
make_context_manager4() as cm4,
):
passBut AFAICT Black only implemented the second piece, and not the continuation escapes. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Implement Black's
wrap_multiple_context_managers_in_parensas a preview style. The new style only applies to Py39+.Gets formatted as
Black automatically detects the python version used in a file based on the syntax used. E.g. a file using match statements is Py310+. I'm unsure if we should autodetect the python version based on the syntax or rely on
requires-pythonThe text was updated successfully, but these errors were encountered: