From 1f71bfc5d2379744c6c21e7b19f972632616091a Mon Sep 17 00:00:00 2001 From: lebaudantoine Date: Wed, 3 Sep 2025 18:43:18 +0200 Subject: [PATCH] =?UTF-8?q?=F0=9F=8E=A8(backend)=20use=20pylint=20error=20?= =?UTF-8?q?names=20instead=20of=20codes=20in=20disable=20comments?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Replace pylint error codes with descriptive error names in disable comments to make suppressed warnings explicit and improve code readability. --- src/backend/core/api/serializers.py | 2 +- src/backend/core/services/livekit_events.py | 2 +- src/backend/core/services/participants_management.py | 2 +- src/backend/core/tests/recording/event/test_authentication.py | 2 +- src/backend/core/tests/recording/event/test_notification.py | 2 +- src/backend/core/tests/recording/event/test_parsers.py | 2 +- .../core/tests/recording/service/test_recording_events.py | 2 +- .../core/tests/recording/test_api_recordings_storage_hook.py | 2 +- src/backend/core/tests/recording/worker/test_factories.py | 2 +- src/backend/core/tests/recording/worker/test_mediator.py | 2 +- src/backend/core/tests/recording/worker/test_services.py | 2 +- src/backend/core/tests/rooms/test_api_rooms_create.py | 2 +- .../core/tests/rooms/test_api_rooms_creation_callback.py | 2 +- src/backend/core/tests/rooms/test_api_rooms_invite.py | 2 +- .../core/tests/rooms/test_api_rooms_participants_management.py | 2 +- src/backend/core/tests/rooms/test_api_rooms_start_recording.py | 2 +- src/backend/core/tests/rooms/test_api_rooms_stop_recording.py | 2 +- src/backend/core/tests/rooms/test_api_rooms_webhook.py | 2 +- 18 files changed, 18 insertions(+), 18 deletions(-) diff --git a/src/backend/core/api/serializers.py b/src/backend/core/api/serializers.py index abb02459..3b0743e4 100644 --- a/src/backend/core/api/serializers.py +++ b/src/backend/core/api/serializers.py @@ -1,6 +1,6 @@ """Client serializers for the Meet core app.""" -# pylint: disable=W0223,E0611 +# pylint: disable=abstract-method,no-name-in-module from django.utils.translation import gettext_lazy as _ diff --git a/src/backend/core/services/livekit_events.py b/src/backend/core/services/livekit_events.py index 9c15b741..980c49c9 100644 --- a/src/backend/core/services/livekit_events.py +++ b/src/backend/core/services/livekit_events.py @@ -1,6 +1,6 @@ """LiveKit Events Service""" -# pylint: disable=E1101 +# pylint: disable=no-member import uuid from enum import Enum diff --git a/src/backend/core/services/participants_management.py b/src/backend/core/services/participants_management.py index 4e07af25..44b633d8 100644 --- a/src/backend/core/services/participants_management.py +++ b/src/backend/core/services/participants_management.py @@ -1,6 +1,6 @@ """Participants management service for LiveKit rooms.""" -# pylint: disable=R0913,E0611,R0917 +# pylint: disable=too-many-arguments,no-name-in-module,too-many-positional-arguments # ruff: noqa:PLR0913 import json diff --git a/src/backend/core/tests/recording/event/test_authentication.py b/src/backend/core/tests/recording/event/test_authentication.py index a207ada1..450b5ed7 100644 --- a/src/backend/core/tests/recording/event/test_authentication.py +++ b/src/backend/core/tests/recording/event/test_authentication.py @@ -2,7 +2,7 @@ Test event authentication. """ -# pylint: disable=E1128 +# pylint: disable=assignment-from-no-return from django.test import RequestFactory diff --git a/src/backend/core/tests/recording/event/test_notification.py b/src/backend/core/tests/recording/event/test_notification.py index 641e081e..856cea52 100644 --- a/src/backend/core/tests/recording/event/test_notification.py +++ b/src/backend/core/tests/recording/event/test_notification.py @@ -2,7 +2,7 @@ Test event notification. """ -# pylint: disable=E1128,W0621,W0613,W0212 +# pylint: disable=assignment-from-no-return,redefined-outer-name,unused-argument,protected-access import datetime import smtplib diff --git a/src/backend/core/tests/recording/event/test_parsers.py b/src/backend/core/tests/recording/event/test_parsers.py index 65904c6c..8316609b 100644 --- a/src/backend/core/tests/recording/event/test_parsers.py +++ b/src/backend/core/tests/recording/event/test_parsers.py @@ -2,7 +2,7 @@ Test event parsers. """ -# pylint: disable=W0212,W0621,W0613 +# pylint: disable=protected-access,redefined-outer-name,unused-argument from unittest import mock diff --git a/src/backend/core/tests/recording/service/test_recording_events.py b/src/backend/core/tests/recording/service/test_recording_events.py index 408f7f70..f878117f 100644 --- a/src/backend/core/tests/recording/service/test_recording_events.py +++ b/src/backend/core/tests/recording/service/test_recording_events.py @@ -2,7 +2,7 @@ Test RecordingEventsService service. """ -# pylint: disable=W0621 +# pylint: disable=redefined-outer-name from unittest import mock diff --git a/src/backend/core/tests/recording/test_api_recordings_storage_hook.py b/src/backend/core/tests/recording/test_api_recordings_storage_hook.py index 73372044..5d923406 100644 --- a/src/backend/core/tests/recording/test_api_recordings_storage_hook.py +++ b/src/backend/core/tests/recording/test_api_recordings_storage_hook.py @@ -2,7 +2,7 @@ Test recordings API endpoints in the Meet core app: save recording. """ -# pylint: disable=W0621,W0613 +# pylint: disable=redefined-outer-name,unused-argument import uuid from unittest import mock diff --git a/src/backend/core/tests/recording/worker/test_factories.py b/src/backend/core/tests/recording/worker/test_factories.py index 2bd448d2..12952ffa 100644 --- a/src/backend/core/tests/recording/worker/test_factories.py +++ b/src/backend/core/tests/recording/worker/test_factories.py @@ -2,7 +2,7 @@ Test worker service factories. """ -# pylint: disable=W0212,W0621,W0613 +# pylint: disable=protected-access,redefined-outer-name,unused-argument from dataclasses import FrozenInstanceError from unittest.mock import Mock diff --git a/src/backend/core/tests/recording/worker/test_mediator.py b/src/backend/core/tests/recording/worker/test_mediator.py index 81fd7904..2bafb317 100644 --- a/src/backend/core/tests/recording/worker/test_mediator.py +++ b/src/backend/core/tests/recording/worker/test_mediator.py @@ -1,6 +1,6 @@ """Test WorkerServiceMediator class.""" -# pylint: disable=W0621,W0613 +# pylint: disable=redefined-outer-name,unused-argument from unittest.mock import Mock diff --git a/src/backend/core/tests/recording/worker/test_services.py b/src/backend/core/tests/recording/worker/test_services.py index ffd9eb18..1591590d 100644 --- a/src/backend/core/tests/recording/worker/test_services.py +++ b/src/backend/core/tests/recording/worker/test_services.py @@ -2,7 +2,7 @@ Test worker service classes. """ -# pylint: disable=W0212,W0621,W0613,E1101 +# pylint: disable=protected-access,redefined-outer-name,unused-argument,no-member from unittest.mock import AsyncMock, Mock, patch diff --git a/src/backend/core/tests/rooms/test_api_rooms_create.py b/src/backend/core/tests/rooms/test_api_rooms_create.py index f5c2412b..3c54d6ae 100644 --- a/src/backend/core/tests/rooms/test_api_rooms_create.py +++ b/src/backend/core/tests/rooms/test_api_rooms_create.py @@ -2,7 +2,7 @@ Test rooms API endpoints in the Meet core app: create. """ -# pylint: disable=W0621,W0613 +# pylint: disable=redefined-outer-name,unused-argument from django.core.cache import cache import pytest diff --git a/src/backend/core/tests/rooms/test_api_rooms_creation_callback.py b/src/backend/core/tests/rooms/test_api_rooms_creation_callback.py index e57730be..8f1d38c4 100644 --- a/src/backend/core/tests/rooms/test_api_rooms_creation_callback.py +++ b/src/backend/core/tests/rooms/test_api_rooms_creation_callback.py @@ -2,7 +2,7 @@ Test rooms API endpoints in the Meet core app: creation callback functionality. """ -# pylint: disable=W0621,W0613 +# pylint: disable=redefined-outer-name,unused-argument from django.core.cache import cache import pytest diff --git a/src/backend/core/tests/rooms/test_api_rooms_invite.py b/src/backend/core/tests/rooms/test_api_rooms_invite.py index 3616b55b..1bc91ede 100644 --- a/src/backend/core/tests/rooms/test_api_rooms_invite.py +++ b/src/backend/core/tests/rooms/test_api_rooms_invite.py @@ -2,7 +2,7 @@ Test rooms API endpoints in the Meet core app: invite. """ -# pylint: disable=W0621,W0613 +# pylint: disable=redefined-outer-name,unused-argument import json import random diff --git a/src/backend/core/tests/rooms/test_api_rooms_participants_management.py b/src/backend/core/tests/rooms/test_api_rooms_participants_management.py index d363460e..647fe007 100644 --- a/src/backend/core/tests/rooms/test_api_rooms_participants_management.py +++ b/src/backend/core/tests/rooms/test_api_rooms_participants_management.py @@ -2,7 +2,7 @@ Test rooms API endpoints in the Meet core app: participants management. """ -# pylint: disable=W0621,W0613, W0212 +# pylint: disable=redefined-outer-name,unused-argument,protected-access import random from unittest import mock diff --git a/src/backend/core/tests/rooms/test_api_rooms_start_recording.py b/src/backend/core/tests/rooms/test_api_rooms_start_recording.py index 92552de6..7b211317 100644 --- a/src/backend/core/tests/rooms/test_api_rooms_start_recording.py +++ b/src/backend/core/tests/rooms/test_api_rooms_start_recording.py @@ -2,7 +2,7 @@ Test rooms API endpoints in the Meet core app: start recording. """ -# pylint: disable=W0621,W0613 +# pylint: disable=redefined-outer-name,unused-argument from unittest import mock diff --git a/src/backend/core/tests/rooms/test_api_rooms_stop_recording.py b/src/backend/core/tests/rooms/test_api_rooms_stop_recording.py index e6319f08..89dc9b93 100644 --- a/src/backend/core/tests/rooms/test_api_rooms_stop_recording.py +++ b/src/backend/core/tests/rooms/test_api_rooms_stop_recording.py @@ -2,7 +2,7 @@ Test rooms API endpoints in the Meet core app: stop recording. """ -# pylint: disable=W0621,W0613 +# pylint: disable=redefined-outer-name,unused-argument from unittest import mock diff --git a/src/backend/core/tests/rooms/test_api_rooms_webhook.py b/src/backend/core/tests/rooms/test_api_rooms_webhook.py index 5c97686f..8de9f8b0 100644 --- a/src/backend/core/tests/rooms/test_api_rooms_webhook.py +++ b/src/backend/core/tests/rooms/test_api_rooms_webhook.py @@ -2,7 +2,7 @@ Test LiveKit webhook endpoint on the rooms API. """ -# pylint: disable=R0913,W0621,R0917,W0613 +# pylint: disable=too-many-arguments,redefined-outer-name,too-many-positional-arguments,unused-argument import base64 import hashlib import json