feat(ci): migrate to ruff for linting

Author: TigerCodeHub

manager/.flake8

@@ -4,15 +4,10 @@ url = "https://pypi.org/simple"
 verify_ssl = true
 
 [dev-packages]
-autopep8 = "~=2.3.2"
-black = "~=26.1.0"
 coverage = "~=7.13.2"
 django-stubs = "~=5.1.0"
-flake8 = "~=7.3.0"
-isort = "~=7.0.0"
 mypy = "~=1.19.1"
-pylint = "~=4.0.4"
-pylint-django = "~=2.7.0"
+ruff = "~=0.16.1"
 
 [packages]
 bleach = "~=6.4.0"

manager/.isort.cfg

@@ -165,7 +165,8 @@ async def site_updated(self, event: Dict[str, Any]) -> None:  # pylint: disable=
         await self.send_site_info()
 
     async def operation_updated(
-        self, event: Dict[str, Any]  # pylint: disable=unused-argument
+        self,
+        event: Dict[str, Any],  # pylint: disable=unused-argument
     ) -> None:
         await self.send_site_info()
 
@@ -381,7 +382,8 @@ async def site_updated(self, event: Dict[str, Any]) -> None:  # pylint: disable=
                 await self.close()
 
     async def operation_updated(
-        self, event: Dict[str, Any]  # pylint: disable=unused-argument
+        self,
+        event: Dict[str, Any],  # pylint: disable=unused-argument
     ) -> None:
         pass
 
@@ -528,7 +530,8 @@ async def site_updated(self, event: Dict[str, Any]) -> None:  # pylint: disable=
                 await self.close_monitor()
 
     async def operation_updated(
-        self, event: Dict[str, Any]  # pylint: disable=unused-argument
+        self,
+        event: Dict[str, Any],  # pylint: disable=unused-argument
     ) -> None:
         pass
 
@@ -631,7 +634,8 @@ async def site_updated(self, event: Dict[str, Any]) -> None:  # pylint: disable=
                 await self.close()
 
     async def operation_updated(
-        self, event: Dict[str, Any]  # pylint: disable=unused-argument
+        self,
+        event: Dict[str, Any],  # pylint: disable=unused-argument
     ) -> None:
         pass
 
@@ -746,7 +750,8 @@ async def site_updated(self, event: Dict[str, Any]) -> None:  # pylint: disable=
                 await self.close()
 
     async def operation_updated(
-        self, event: Dict[str, Any]  # pylint: disable=unused-argument
+        self,
+        event: Dict[str, Any],  # pylint: disable=unused-argument
     ) -> None:
         pass
 

manager/Pipfile

@@ -227,9 +227,11 @@ class DatabaseCreateForm(forms.Form):
 
 class ImageSelectForm(forms.Form):
     image = forms.ChoiceField(
-        choices=lambda: DockerImage.objects.filter_user_visible()  # type: ignore
-        .order_by("friendly_name")
-        .values_list("name", "friendly_name"),
+        choices=lambda: (
+            DockerImage.objects.filter_user_visible()  # type: ignore
+            .order_by("friendly_name")
+            .values_list("name", "friendly_name")
+        ),
         required=True,
         widget=forms.widgets.RadioSelect(),
     )

manager/Pipfile.lock

@@ -374,9 +374,9 @@
 
 CSRF_COOKIE_SECURE = SESSION_COOKIE_SECURE = not DEBUG
 
-assert len(DIRECTOR_APPSERVER_HOSTS) == len(
-    DIRECTOR_APPSERVER_WS_HOSTS
-), "DIRECTOR_APPSERVER_HOSTS and DIRECTOR_APPSERVER_WS_HOSTS must be the same length"
+assert len(DIRECTOR_APPSERVER_HOSTS) == len(DIRECTOR_APPSERVER_WS_HOSTS), (
+    "DIRECTOR_APPSERVER_HOSTS and DIRECTOR_APPSERVER_WS_HOSTS must be the same length"
+)
 
 # We allow string memory limits in the model, but that's because we can do validation there.
 # If we allowed strings here, then a typo while editing secret.py could make nearly all sites on