Enable ruff PL (pylint) rules

- Add targeted ignores for API model patterns (too many args, lazy imports, many returns)
- Allow magic values in tests and complex logic in utils
- Suppress PLR2004 for HTTP status code and JWT segment comparisons

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

@@ -728,7 +728,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

@@ -122,16 +122,21 @@ select = [
     "PTH",
     # flake8-builtins
     "A",
+    # pylint
+    "PL",
 ]
 ignore = [
     "E501",    # Line too long
     "TRY003",  # Long exception messages are acceptable for domain-specific errors
     "A002",    # Shadowing builtins (id, type) is intentional for API field names
+    "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
 ]
 
 [tool.ruff.lint.per-file-ignores]
-"tests/**" = ["S101", "SLF001"]  # Allow assert and private member access in tests
-"utils/**" = ["INP001", "BLE001"]  # Utils are scripts, not packages
+"tests/**" = ["S101", "SLF001", "PLR2004"]  # Allow assert, private member access, and magic values in tests
+"utils/**" = ["INP001", "BLE001", "PLR0912", "PLR0915"]  # Utils are scripts, not packages; complex generators acceptable
 
 [tool.ruff.lint.pydocstyle]
 convention = "google"  # Accepts: "google", "numpy", or "pep257".