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