Skip to content

FURB105 fix removes sep arguments with side effects #13126

@dscorbett

Description

@dscorbett

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

Metadata

Metadata

Assignees

Labels

acceptedReady for implementationbugSomething isn't workingfixesRelated to suggested fixes for violations

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions