Function contains unused argument PYL-W0613
Anti-pattern
Major
2 years ago
—
4 years old
Unused argument 'exc'
167 db.connect(reuse_if_open=True)
168
169 @app.teardown_request
170 def db_close(exc: Any) -> None:171 if not db.is_closed():
172 db.close()
173
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