-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Open
Labels
F: linebreakHow should we split up lines?How should we split up lines?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 style
tests = [
(
BusinessHour(),
{
Timestamp("2014-07-04 15:00")
+ Nano(5): this_is_a_very_long_function("2014-07-04 16:00"),
Timestamp("2014-07-04 16:00")
+ Nano(5): this_is_a_very_long_function("2014-07-04 16:00"),
Timestamp("2014-07-04 16:00")
- Nano(5): this_is_a_very_long_function("2014-07-04 16:00"),
},
),
]Desired style
tests = [
(
BusinessHour(),
{
Timestamp("2014-07-04 15:00") + Nano(5): (
this_is_a_very_long_function("2014-07-04 16:00")
),
Timestamp("2014-07-04 16:00") + Nano(5): (
this_is_a_very_long_function("2014-07-04 16:00"),
),
Timestamp("2014-07-04 16:00") - Nano(5): (
this_is_a_very_long_function("2014-07-04 16:00")
),
},
),
]Additional context
This is an example found in #3440 while working on the dict value. I think the issue is that we don't treat : as a delimiter thus the "operators" are broken instead of :.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
F: linebreakHow should we split up lines?How should we split up lines?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?