🚨(lint) lint code with pylint 3.3.0

New pylint version 3.3.0 has new rule and code must be fixed.
openfun · Oct 21, 2024 · a99a9c4 · a99a9c4
1 parent acd49d1
commit a99a9c4
Show file tree

Hide file tree

Showing 6 changed files with 6 additions and 2 deletions.
diff --git a/src/django_peertube_runner_connector/socketio/manager.py b/src/django_peertube_runner_connector/socketio/manager.py
@@ -69,6 +69,7 @@ class AsyncSentinelRedisManager(AsyncRedisManager):
 
     name = "aioredis"
 
+    # pylint: disable=too-many-positional-arguments
     def __init__(
         self,
         sentinels=None,

diff --git a/src/django_peertube_runner_connector/utils/job_handlers/abstract_job_handler.py b/src/django_peertube_runner_connector/utils/job_handlers/abstract_job_handler.py
@@ -32,7 +32,7 @@ class AbstractJobHandler(ABC):
     def create(self, *args, **kwargs):
         """This method should be implemented by subclasses."""
 
-    # pylint: disable=too-many-arguments
+    # pylint: disable=too-many-arguments,too-many-positional-arguments
     def create_runner_job(
         self,
         domain: str,

diff --git a/src/django_peertube_runner_connector/utils/job_handlers/vod_hls_transcoding_job_handler.py b/src/django_peertube_runner_connector/utils/job_handlers/vod_hls_transcoding_job_handler.py
@@ -33,7 +33,7 @@
 class VODHLSTranscodingJobHandler(AbstractVODTranscodingJobHandler):
     """Handler for vod hls transcoding jobs."""
 
-    # pylint: disable=arguments-differ
+    # pylint: disable=arguments-differ,too-many-positional-arguments
     def create(self, video: Video, resolution, fps, depends_on_runner_job, domain: str):
         job_uuid = uuid.uuid4()
 

diff --git a/src/django_peertube_runner_connector/utils/transcoding/job_creation.py b/src/django_peertube_runner_connector/utils/transcoding/job_creation.py
@@ -72,6 +72,7 @@ def get_closest_framerate_standard(fps, fps_setting):
     return sorted(setting_transcoding[fps_setting], key=lambda x: fps % x)[0]
 
 
+# pylint: disable=too-many-positional-arguments
 def build_lower_resolution_job_payloads(
     video: Video,
     input_video_resolution,

diff --git a/tests/tests_django_peertube_runner_connector/test_transcode.py b/tests/tests_django_peertube_runner_connector/test_transcode.py
@@ -57,6 +57,7 @@ def test_transcode(self, mock_process):
     @patch("django_peertube_runner_connector.transcode.get_video_stream_duration")
     @patch("django_peertube_runner_connector.transcode.build_video_thumbnails")
     @patch("django_peertube_runner_connector.transcode.create_transcoding_jobs")
+    # pylint: disable=too-many-positional-arguments
     def test_process_transcoding(
         self, mock_transcoding, mock_thumbnails, mock_duration, mock_build, mock_probe
     ):

diff --git a/...ango_peertube_runner_connector/utils/job_handlers/test_vod_hls_transcoding_job_handler.py b/...ango_peertube_runner_connector/utils/job_handlers/test_vod_hls_transcoding_job_handler.py
@@ -93,6 +93,7 @@ def test_create(self):
         "django_peertube_runner_connector.utils.job_handlers."
         "vod_hls_transcoding_job_handler.generate_hls_video_filename",
     )
+    # pylint: disable=too-many-positional-arguments
     def test_specific_complete(
         self,
         mock_generate_hls,