Function contains unused argument PYL-W0613
Anti-pattern
a year ago
—
a year old
Unused argument 'hostname'
10
11class HelmChartRepoSession(ProjectHolder):
12
13 def __init__(self, repo, hostname=None):14 super(HelmChartRepoSession, self).__init__()
15 if not repo.endswith('Chart.yaml'):
16 repo = repo.rstrip('/') + '/Chart.yaml'Unused argument 'hostname'
9class SystemRepoSession(ProjectHolder):
10 """Version holder based on system package repositories."""
11
12 def __init__(self, repo, hostname=None): 13 super(SystemRepoSession, self).__init__()
14 self.set_repo(repo)
15
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