Function contains unused argument PYL-W0613
Anti-pattern
Major
2 months ago
—
3 months old
Unused argument 'allow_changes'
1446 return base_provenance
1447
1448
1449def prepare_dump(data: IOData, allow_changes: bool, filename: str) -> IOData:1450 """Check the compatibility of the IOData object with QCScheme.
1451
1452 Parameters
Description
An unused argument can lead to confusions. It should be removed. If this variable is necessary, name the variable _ or start the name with unused or _unused.
Bad practice
def square(x, y=1):
return x * x
class MySubClass(MyClass):
def __init__(self, number):
self.value = 42 # argument `number` remains unused
Preferred:
def square(x):
return x * x
class MySubClass(MyClass):
def __init__(self, _):
self.value = 42