-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Closed
Labels
F: parenthesesToo many parentheses, not enough parentheses, and so on.Too many parentheses, not enough parentheses, and so on.S: acceptedThe changes in this design / enhancement issue have been accepted and can be implementedThe changes in this design / enhancement issue have been accepted and can be implementedT: styleWhat do we want Blackened code to look like?What do we want Blackened code to look like?
Description
Describe the style change
Examples in the current Black future style
items = [
(
{"key1": "val1", "key2": "val2", "key3": "val3"}
if some_var == "longstring"
else {"key": "val"}
)
]
Desired style
items = [
{'key1': 'val1', 'key2': 'val2', 'key3': 'val3'}
if some_var == 'longstring'
else {'key': 'val'}
]
Additional context
This is a change from #2278.
I think for this edge case, it shouldn't be parenthesized which also adds an extra indentation level.
calumy, aaossa and mariuswallraff
Metadata
Metadata
Assignees
Labels
F: parenthesesToo many parentheses, not enough parentheses, and so on.Too many parentheses, not enough parentheses, and so on.S: acceptedThe changes in this design / enhancement issue have been accepted and can be implementedThe changes in this design / enhancement issue have been accepted and can be implementedT: styleWhat do we want Blackened code to look like?What do we want Blackened code to look like?