asottile/add-trailing-comma

autopep8 and add-trailing-comma fight over indentation

chriskuehl opened this issue · 1 comments

Here's an (admittedly poorly-formatted) reproduction. FWIW I prefer add-trailing-comma's approach, but curious to hear your thoughts. This is obviously weird indentation but ideally they wouldn't fight. Do we need to make autopep8 smarter?

what add-trailing-comma wants

[a()
    for b in c
    if (
        d
    )
]

what autopep8 wants

[a()
    for b in c
    if (
        d
)
]

I kind of expect this to happen too:

-[a()
+[
+    a()
     for b in c
     if (
         d
     )
 ]

I'll see what I can do to make that so