Merge pull request road-core#148 from tisnik/update-ruff-to-version-0.8.0

Update Ruff to version 0.8.0

Author: tisnik

ols/app/main.py

@@ -75,9 +75,8 @@ async def log_requests_responses(
 ) -> Response:
     """Middleware for logging of HTTP requests and responses, at debug level."""
     # Bail out early if not logging or Prometheus metrics logging is suppressed
-    if (
-        not logger.isEnabledFor(logging.DEBUG)
-        or config.ols_config.logging_config.suppress_metrics_in_log
+    if not logger.isEnabledFor(logging.DEBUG) or (
+        config.ols_config.logging_config.suppress_metrics_in_log
         and request.url.path == "/metrics"
     ):
         return await call_next(request)

ols/src/cache/redis_cache.py

@@ -8,7 +8,7 @@
 from redis.backoff import ExponentialBackoff
 from redis.exceptions import (
     BusyLoadingError,
-    ConnectionError,
+    ConnectionError,  # noqa: A004
     RedisError,
 )
 from redis.retry import Retry

pdm.lock

@@ -1,12 +1,12 @@
 [tool.ruff]
 
 # description of all rules are available on https://docs.astral.sh/ruff/rules/
-lint.select = ["D", "E", "F", "W", "C", "S", "I", "TCH", "SLOT", "RUF", "C90", "N", "YTT", "ASYNC", "A", "C4", "T10", "PGH"]
+lint.select = ["D", "E", "F", "W", "C", "S", "I", "TCH", "SLOT", "RUF", "C90", "N", "YTT", "ASYNC", "A", "C4", "T10", "PGH", "FURB", "PERF", "AIR", "NPY", "FLY"]
 
 # we need to check 'mood' of all docstrings, this needs to be enabled explicitly
 lint.extend-select = ["D401"]
 
-lint.ignore = ["RUF021", "A004"]
+lint.ignore = []
 
 target-version = "py311"
 lint.pydocstyle.convention = "google"
@@ -46,7 +46,7 @@ dev = [
     "pytest-asyncio==0.24.0",
     "pydantic==2.9.1",
     "rouge-score==0.1.2",  # Required for model evaluation
-    "ruff==0.7.0",
+    "ruff==0.8.0",
     "bandit==1.7.9",
     "types-requests==2.32.0.20240622",
     "gradio==4.44.0",

pyproject.toml

@@ -173,7 +173,7 @@ def get_pod_by_prefix(
         result = get_pods(namespace)
         for pod in result:
             if prefix in pod:
-                pods.append(pod)
+                pods.append(pod)  # noqa: PERF401
         if fail_not_found and not pods:
             assert False, f"No OLS api server pod found in list pods: {result}"
         return pods