Black inconsistent about arguments with many alternative types

[Illviljan]

Describe the style change
Black is not consistent when arguments can be many types.

This is the original format, both stat_lengthh and stat_lengths have the same typing but one has parenthesis around it:

from __future__ import annotations

from typing import Any, Mapping


class test:
    def pad(
        self,
        stat_lengthh: (int | tuple[int, int] | Mapping[Any, tuple[int, int]] | None) = None,
        stat_lengths: int | tuple[int, int] | Mapping[Any, tuple[int, int]] | None = None,
    ):
        pass

Examples in the current Black style

Black formats the same typing in two different ways making it hard to see they are exactly the same:

from __future__ import annotations

from typing import Any, Mapping


class test:
    def pad(
        self,
        stat_lengthh: (
            int | tuple[int, int] | Mapping[Any, tuple[int, int]] | None
        ) = None,
        stat_lengths: int
        | tuple[int, int]
        | Mapping[Any, tuple[int, int]]
        | None = None,
    ):
        pass

Desired style
I prefer the variant with parenthesis, the extra indent makes it easier to tell where the new arguments are.

from __future__ import annotations

from typing import Any, Mapping


class test:
    def pad(
        self,
        stat_lengthh: (
            int | tuple[int, int] | Mapping[Any, tuple[int, int]] | None
        ) = None,
        stat_lengths: (
            int | tuple[int, int] | Mapping[Any, tuple[int, int]] | None
        ) = None,
    ):
        pass

Additional context

[felix-hilden]

All for the consistency, but personally I'm not sure which style would be better. Thanks for reporting!

[JelleZijlstra]

I also prefer the parenthesized version, since it makes it easier to see where each new parameter starts.

Anyway, this is a duplicate of #2316.