Too much whitespace when last or only param to a function/method call is a payload dict func({...})

[fmalina]

Describe the style change

I'd like black use more terse formatting when it comes across dictionaries as params in function and method calls. Here's a django function based view example, function calls accepting payload dicts end up looking wrong

Example in the current Black style

def view(request):
    ...
    return render(
        request,
        'places/detail.html',
        {
            'place': place,
            'opening_times': times,
            'photos': photos,
            'staff': staff,
            'town': town,
            'segment': sector,
        },
    )

Desired style

def view(request):
    ...
    return render(request, 'places/detail.html', {
        'place': place,
        'opening_times': times,
        'photos': photos,
        'staff': staff,
        'town': town,
        'segment': sector
    })

[felix-hilden]

Hi! How would you suggest we format calls with an argument after an exploded collection? I'd argue it should take the current form. But you could make a case for the suggested formatting only if the last argument is exploded.

[fmalina]

I can't imagine the former without an example, but I would agree with the latter

[felix-hilden]

This I would expect to stay as it is.

def view(request):
    ...
    return render(
        request,
        'places/detail.html',
        {...},
        some_other_argument,  # after collection to be exploded
    )

I'm open to making an exception for the last argument, but I'm quite happy with our current formatting as well. Thoughts by others?

[fmalina]

I have just had a super rare case like you show above where dictionary argument was followed by another and I went with current black style formatting too

[felix-hilden]

I think this is sufficiently similar to #1811 so that we can close this. I'll include your case in the discussion as well. Hopefully we'll get to resolving it one way or the other!