diff options
author | Pierre Sassoulas <pierre.sassoulas@gmail.com> | 2021-11-23 18:44:18 +0100 |
---|---|---|
committer | Pierre Sassoulas <pierre.sassoulas@gmail.com> | 2021-11-23 18:44:42 +0100 |
commit | 82d7b803aafe24bca42e393d66782bca297f9742 (patch) | |
tree | 2edc68c34a84d99d8d9b7669b42bfb3c94c9de31 | |
parent | d5dec5c7053c855452091f94b555c7d942c0d77e (diff) | |
download | pylint-git-82d7b803aafe24bca42e393d66782bca297f9742.tar.gz |
Remove disable for else-if-used in pylint/checkers/classes.py
-rw-r--r-- | pylint/checkers/classes.py | 26 |
1 files changed, 12 insertions, 14 deletions
diff --git a/pylint/checkers/classes.py b/pylint/checkers/classes.py index 319a37f73..37651cdfd 100644 --- a/pylint/checkers/classes.py +++ b/pylint/checkers/classes.py @@ -1848,20 +1848,18 @@ a metaclass class method.", "bad-mcs-method-argument", node.name, ) - # regular class - else: # pylint: disable=else-if-used - # class method - if node.type == "classmethod" or node.name == "__class_getitem__": - self._check_first_arg_config( - first, - self.config.valid_classmethod_first_arg, - node, - "bad-classmethod-argument", - node.name, - ) - # regular method without self as argument - elif first != "self": - self.add_message("no-self-argument", node=node) + # regular class with class method + elif node.type == "classmethod" or node.name == "__class_getitem__": + self._check_first_arg_config( + first, + self.config.valid_classmethod_first_arg, + node, + "bad-classmethod-argument", + node.name, + ) + # regular class with regular method without self as argument + elif first != "self": + self.add_message("no-self-argument", node=node) def _check_first_arg_config(self, first, config, node, message, method_name): if first not in config: |