FURB105 [*] Unnecessary separator passed to print

[SHEscher]

Hi, there seems to be a mismatch:

# print each element of a list to a new line
print(*a_list_with_elements, sep="\n")

ruff . --select FURB105  --preview
# out: ... FURB105 [*] Unnecessary separator passed to `print`

However, FURB105 apparently checks for empty prints:

ruff rule FURB105
# out: Checks for `print` calls with an empty string as the only positional ...

Version ruff 0.1.1

[charliermarsh]

This seems like a bug, since the separator is necessary here.

[SHEscher]

For

print(*a_list_with_elements, sep=",")

it is the same behaviour.

Plus description of rule and application don't seem to match, since it is not an empty print()

[charliermarsh]

Yeah the rule is slightly more expansive than the description suggests -- I've updated it. It also looks for print calls with one or fewer arguments that define a custom separator, since the separator is redundant in that case. I've also fixed the bug here related to starred arguments.