From 736d0c06f46f514e10ab2b52c7c6947b4a1cf209 Mon Sep 17 00:00:00 2001 From: Jim Lancaster <40281233+zoltan57@users.noreply.github.com> Date: Sun, 23 Aug 2026 18:19:05 -0500 Subject: [PATCH] chore: enable logging format lint rule Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> --- ruff.toml | 1 + src/transcription/services/workflows.py | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/ruff.toml b/ruff.toml index 4b97024..34dfe99 100644 --- a/ruff.toml +++ b/ruff.toml @@ -26,6 +26,7 @@ extend-select = [ "E", "W", # https://docs.astral.sh/ruff/rules/#pycodestyle-e-w "F", # https://docs.astral.sh/ruff/rules/#pyflakes-f "FURB", # https://docs.astral.sh/ruff/rules/#refurb-furb + "G", # https://docs.astral.sh/ruff/rules/#flake8-logging-format-g "I", # https://docs.astral.sh/ruff/rules/#isort-i "N", # https://docs.astral.sh/ruff/rules/#pep8-naming-n "PD", # https://docs.astral.sh/ruff/rules/#pandas-vet-pd diff --git a/src/transcription/services/workflows.py b/src/transcription/services/workflows.py index 37195a7..58cc2fe 100644 --- a/src/transcription/services/workflows.py +++ b/src/transcription/services/workflows.py @@ -190,7 +190,7 @@ async def advance_job( session=session, ) else: - logger.error(f"Job {job.id} has failed and reached max retries.") + logger.error("Job %s has failed and reached max retries.", job.id) return case _: return @@ -207,7 +207,7 @@ async def process_queued_job( # noqa: PLR0915 runtime_settings = settings or get_settings() current_status = _coerce_job_status(job.status) if current_status not in {JobStatus.QUEUED, JobStatus.PROCESSING}: - logger.warning(f"Job {job.id} is not queued. Current status: {job.status}") + logger.warning("Job %s is not queued. Current status: %s", job.id, job.status) return # Transaction A: claim job for processing. Reached only when a caller hands us a