fix(linter): pylint fixes in python

Signed-off-by: AtomicFS <vojtech.vesely@9elements.com>
9elements · Nov 18, 2024 · bbcbe25 · bbcbe25
1 parent 9216098
commit bbcbe25
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 1 deletion.
diff --git a/.dagger-ci/daggerci/lib/orchestrator.py b/.dagger-ci/daggerci/lib/orchestrator.py
@@ -348,6 +348,7 @@ async def __build__(
         #   "AttributeError: module 'dagger' has no attribute 'api'"
 
         # pylint: disable=too-many-arguments
+        # pylint: disable=too-many-positional-arguments
         """
         Does the actual building of docker container
         """

diff --git a/.dagger-ci/daggerci/lib/results.py b/.dagger-ci/daggerci/lib/results.py
@@ -24,7 +24,7 @@ def __init__(self) -> None:
         self.results: dict[str, dict[str, dict[str, str | bool | None]]] = {}
 
     def add(
-        self,  # pylint: disable=too-many-arguments
+        self,
         top_element: str,
         dockerfile: str,
         stage: str,
@@ -34,6 +34,10 @@ def add(
         """
         Add entry into results
         """
+
+        # pylint: disable=too-many-arguments
+        # pylint: disable=too-many-positional-arguments
+
         # Update list of all dockerfiles
         if [top_element, dockerfile] not in self.all_dockerfiles:
             self.all_dockerfiles.append([top_element, dockerfile])

diff --git a/.dagger-ci/daggerci/tests/test_orchestrator.py b/.dagger-ci/daggerci/tests/test_orchestrator.py
@@ -1,6 +1,7 @@
 # pylint: disable=missing-function-docstring
 # pylint: disable=missing-module-docstring
 # pylint: disable=too-many-arguments
+# pylint: disable=too-many-positional-arguments
 # pylint: disable=unused-import
 # mypy: disable-error-code="import, no-untyped-def"