Inconsistent method chaining

[rvinzent]

black currently formats method chaining inconsistently. It should always produce the more readable option.

Examples in the current Black style
This is ugly and hard to read:

This occurs only sometimes

def get_queryset(self):
    return SomeModel.objects.filter(
        related__field=self.kwargs["field"]
    ).prefetch_related("one_field")

With longer method chaining, the more readable format is generated:

    def get_queryset(self):
        return (
            SomeModel.objects.filter(related__field=self.kwargs["field"])
            .prefetch_related("one_field")
            .select_related()
        )

Desired style:
It would be nice if the more readable format was always produced in the first case above. It would be easier to read and more consistent.

def get_queryset(self):
    return (
        SomeModel.objects
        .filter(related__field=self.kwargs["field"])
        .prefetch_related("one_field")
    )

On a related note, sometimes method chains are broken up in a way that harms readability, the suggested format above would be auto-formatted to a less readable version below

def get_queryset(self):
    return SomeModel.objects.filter(related__field=self.kwargs["field"].prefetch_related(
        "one_field"
    )

The desired format would be the same in this case, as it is clearly more readable to have arguments appear next to the functions they are passed to.

[felix-hilden]

This is being discussed in #571, so I'll close this as a duplicate.

[rvinzent]

This is being discussed in #571, so I'll close this as a duplicate.

Ah sorry for missing that. Is there any recent movement on the issue?

[felix-hilden]

No worries! Yes there has been.