Enable ruff PL (pylint) rules with targeted ignores

Add pylint rule set. Globally ignore PLR0913 (too many args in model
constructors), PLC0415 (intentional lazy imports), PLR0911 (factory
return statements). Ignore PLR2004 in tests and PLR0912/PLR0915 in
utils. Add noqa for well-known HTTP status codes and JWT checks.

Author: iMicknl

pyoverkiz/auth/strategies.py

@@ -116,7 +116,7 @@ async def _post_login(self, data: Mapping[str, Any]) -> None:
                 )
 
             # A 204 No Content response cannot have a body, so skip JSON parsing.
-            if response.status == 204:
+            if response.status == 204:  # noqa: PLR2004
                 return
 
             result = await response.json()
@@ -419,7 +419,7 @@ def auth_headers(self, path: str | None = None) -> Mapping[str, str]:
 def _decode_jwt_payload(token: str) -> dict[str, Any]:
     """Decode the payload of a JWT token."""
     parts = token.split(".")
-    if len(parts) < 2:
+    if len(parts) < 2:  # noqa: PLR2004
         raise InvalidTokenError("Malformed JWT received.")
 
     payload_segment = parts[1]

pyoverkiz/client.py

@@ -731,7 +731,7 @@ async def _delete(self, path: str) -> None:
     async def _parse_response(response: ClientResponse) -> Any:
         """Check response status and parse JSON body (returns None for 204)."""
         await check_response(response)
-        if response.status == 204:
+        if response.status == 204:  # noqa: PLR2004
             return None
         return await response.json()
 

pyoverkiz/response_handler.py

@@ -124,7 +124,7 @@ async def check_response(response: ClientResponse) -> None:
         if "is down for maintenance" in result:
             raise MaintenanceError("Server is down for maintenance") from error
 
-        if response.status == 503:
+        if response.status == 503:  # noqa: PLR2004
             raise ServiceUnavailableError(result) from error
 
         raise OverkizError(

pyproject.toml

@@ -73,11 +73,19 @@ select = [
     "C4",
     # pep8-naming
     "N",
+    # pylint
+    "PL",
+]
+ignore = [
+    "E501",     # Line too long
+    "PLR0913",  # Too many arguments — expected for API model constructors
+    "PLC0415",  # Import not at top level — intentional lazy imports
+    "PLR0911",  # Too many return statements — acceptable in factory functions
 ]
-ignore = ["E501"] # Line too long
 
 [tool.ruff.lint.per-file-ignores]
-"tests/**" = ["S101"]  # Allow assert in tests
+"tests/**" = ["S101", "PLR2004"]  # Allow assert and magic values in tests
+"utils/**" = ["PLR0912", "PLR0915"]  # Complex generator scripts are acceptable
 
 [tool.ruff.lint.pydocstyle]
 convention = "google"  # Accepts: "google", "numpy", or "pep257".