fix: Remove disabling no-self-use lint check

python/legion_linux/legion_linux/legion.py

@@ -165,11 +165,9 @@ def _notify(self):
         for func in self.callbacks:
             func(self)
 
-    # pylint: disable=no-self-use
     def exists(self):
         return True
 
-    # pylint: disable=no-self-use
     def get_values(self) -> List[NamedValue]:
         return []
 
@@ -299,7 +297,6 @@ def _find_by_file_pattern(pattern):
                 return matches[0]
         return None
 
-    # pylint: disable=no-self-use
     def get_values(self) -> List[NamedValue]:
         return []
 
@@ -667,7 +664,6 @@ def _exec_cmd(self, cmd, timeout=None) -> Tuple[str, int]:
     def name(self):
         return type(self).__name__
 
-    # pylint: disable=no-self-use
     def get_values(self) -> List[NamedValue]:
         return []
 
@@ -1050,7 +1046,6 @@ def get_settings(self) -> Settings:
             settings.setting_entries[name] = value
         return settings
 
-    # pylint: disable=no-self-use
     def apply_settings(self, preset: Settings):
         for name, value in preset.setting_entries.items():
             log.error("Try seting %s from preset to %s", name, value)

python/legion_linux/legion_linux/legion_cli.py

@@ -66,19 +66,15 @@ def command_disable_cli(self, **_) -> int:
             return self.command_disable()
         return -10
 
-    # pylint: disable=no-self-use
     def exists(self) -> bool:
         return False
 
-    # pylint: disable=no-self-use
     def command_status(self, **_) -> int:
         return 0
 
-    # pylint: disable=no-self-use
     def command_enable(self, **_) -> int:
         return -1
 
-    # pylint: disable=no-self-use
     def command_disable(self, **_) -> int:
         return -1
 

python/legion_linux/pylintrc

@@ -2,7 +2,7 @@
 init-hook='import sys; sys.path.append("python/legion_linux")'
 
 [MESSAGES CONTROL]
-disable=missing-docstring,no-self-use,too-many-arguments,too-many-locals, import-error, duplicate-code, missing-module-docstring, too-many-instance-attributes, too-many-public-methods, no-name-in-module
+disable=missing-docstring,too-many-arguments,too-many-locals, import-error, duplicate-code, missing-module-docstring, too-many-instance-attributes, too-many-public-methods, no-name-in-module
 
 [FORMAT]
 max-line-length=120

tests/test_python.sh

@@ -2,4 +2,4 @@
 set -ex
 DIR=$( cd -- "$( dirname -- "${BASH_SOURCE[0]}" )" &> /dev/null && pwd )
 
-pylint --ignore-patterns=legion_gui_qt6.py,legion_version.py --disable no-self-use --rcfile ${DIR}/../python/legion_linux/pylintrc ${DIR}/../python/legion_linux/legion_linux
+pylint --ignore-patterns=legion_gui_qt6.py,legion_version.py --rcfile ${DIR}/../python/legion_linux/pylintrc ${DIR}/../python/legion_linux/legion_linux