Skip to content

Commit

Permalink
Require named arguments to fix pylint too-many-positional-arguments
Browse files Browse the repository at this point in the history
  • Loading branch information
clelange committed Oct 4, 2024
1 parent 79091d0 commit f165800
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions hepdata_lib/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ def __init__(self):
self.files_to_copy = []
self.additional_resources = []

def add_additional_resource(self, description, location, copy_file=False, file_type=None,
def add_additional_resource(self, description, location, *, copy_file=False, file_type=None,
resource_license=None):
"""
Add any kind of additional resource.
Expand Down Expand Up @@ -143,7 +143,7 @@ class Variable:
# pylint: disable=too-many-instance-attributes
# Eight is reasonable in this case.

def __init__(self, name, is_independent=True, is_binned=True, units="", values=None,
def __init__(self, name, *, is_independent=True, is_binned=True, units="", values=None,
zero_uncertainties_warning=True):
# pylint: disable=too-many-arguments
self.name = name
Expand Down

0 comments on commit f165800

Please sign in to comment.