-
Notifications
You must be signed in to change notification settings - Fork 2k
FURB105 fix removes sep arguments with side effects #13126
Copy link
Copy link
Closed
Labels
acceptedReady for implementationReady for implementationbugSomething isn't workingSomething isn't workingfixesRelated to suggested fixes for violationsRelated to suggested fixes for violations
Description
The fix for FURB105 removes unneeded sep arguments even when they might have side effects. In that case, the fix should be omitted or marked unsafe.
$ ruff --version
ruff 0.6.2
$ cat furb105.py
print(sep=print("sep"))
$ python furb105.py
sep
$ ruff check --isolated --select FURB105 furb105.py --fix
Found 1 error (1 fixed, 0 remaining).
$ cat furb105.py
print()
$ python furb105.py
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
acceptedReady for implementationReady for implementationbugSomething isn't workingSomething isn't workingfixesRelated to suggested fixes for violationsRelated to suggested fixes for violations