Restrict "params" to list or None

[radarhere]

Suggestion for python-pillow#7407

In your PR at the moment,

def cmd_nmake(
    makefile: str | None = None,
    target: str = "",
    params: str | list[str] | tuple[str, ...] = None,
) -> str:
    if params is None:
        params = ""
    elif isinstance(params, (list, tuple)):
        params = " ".join(params)
    else:
        params = str(params)

params is either a string, list or tuple. When it is a string, str(params) is called? I find this odd, since it is already a string.

cmd_nmake is only called either without the third argument, or with a list, so maybe the simplest solution is just to restrict it to those two options?

[hugovk]

Yup, nice cleanup!