From 891df447bbdc575a91771e7a812edbf5de3d1968 Mon Sep 17 00:00:00 2001 From: Adam Moussa Date: Thu, 18 Jun 2026 16:21:34 -0400 Subject: [PATCH] fix(secrev): doc-drift fixture py ruff-clean (root CI runs check + format --check) The repo-root CI lint runs both 'ruff check .' and 'ruff format --check .' over all fixtures. Fixed E701 one-liners and ruff-formatted the sample-service .py files (handlers/*, feature_*.py). Fixture content is irrelevant to doc-drift (keys on file/dir presence + git staleness). --- .../fixtures/doc-drift/clean-repo/handlers/ingest/app.py | 3 ++- .../fixtures/doc-drift/drift-omits-repo/handlers/charge/app.py | 3 ++- .../doc-drift/drift-omits-repo/notifier-service/main.py | 3 ++- .../doc-drift/drift-omits-repo/payments-service/main.py | 3 ++- .../doc-drift/drift-stale-repo/worker-service/feature_1.py | 3 ++- .../doc-drift/drift-stale-repo/worker-service/feature_2.py | 3 ++- .../doc-drift/drift-stale-repo/worker-service/feature_3.py | 3 ++- .../doc-drift/drift-stale-repo/worker-service/feature_4.py | 3 ++- .../doc-drift/drift-stale-repo/worker-service/feature_5.py | 3 ++- .../fixtures/doc-drift/drift-stale-repo/worker-service/main.py | 3 ++- .../fixtures/doc-drift/no-readme-repo/some-service/main.py | 3 ++- 11 files changed, 22 insertions(+), 11 deletions(-) diff --git a/security-review/checkers/fixtures/doc-drift/clean-repo/handlers/ingest/app.py b/security-review/checkers/fixtures/doc-drift/clean-repo/handlers/ingest/app.py index 1f86368..b473d36 100644 --- a/security-review/checkers/fixtures/doc-drift/clean-repo/handlers/ingest/app.py +++ b/security-review/checkers/fixtures/doc-drift/clean-repo/handlers/ingest/app.py @@ -1 +1,2 @@ -def handler(event, ctx): return {} +def handler(event, ctx): + return {} diff --git a/security-review/checkers/fixtures/doc-drift/drift-omits-repo/handlers/charge/app.py b/security-review/checkers/fixtures/doc-drift/drift-omits-repo/handlers/charge/app.py index 1f86368..b473d36 100644 --- a/security-review/checkers/fixtures/doc-drift/drift-omits-repo/handlers/charge/app.py +++ b/security-review/checkers/fixtures/doc-drift/drift-omits-repo/handlers/charge/app.py @@ -1 +1,2 @@ -def handler(event, ctx): return {} +def handler(event, ctx): + return {} diff --git a/security-review/checkers/fixtures/doc-drift/drift-omits-repo/notifier-service/main.py b/security-review/checkers/fixtures/doc-drift/drift-omits-repo/notifier-service/main.py index 9a81d15..af2e9f2 100644 --- a/security-review/checkers/fixtures/doc-drift/drift-omits-repo/notifier-service/main.py +++ b/security-review/checkers/fixtures/doc-drift/drift-omits-repo/notifier-service/main.py @@ -1 +1,2 @@ -class Notifier: pass +class Notifier: + pass diff --git a/security-review/checkers/fixtures/doc-drift/drift-omits-repo/payments-service/main.py b/security-review/checkers/fixtures/doc-drift/drift-omits-repo/payments-service/main.py index ccb1aee..2ffacb2 100644 --- a/security-review/checkers/fixtures/doc-drift/drift-omits-repo/payments-service/main.py +++ b/security-review/checkers/fixtures/doc-drift/drift-omits-repo/payments-service/main.py @@ -1 +1,2 @@ -class Payments: pass +class Payments: + pass diff --git a/security-review/checkers/fixtures/doc-drift/drift-stale-repo/worker-service/feature_1.py b/security-review/checkers/fixtures/doc-drift/drift-stale-repo/worker-service/feature_1.py index b6a829f..64876dd 100644 --- a/security-review/checkers/fixtures/doc-drift/drift-stale-repo/worker-service/feature_1.py +++ b/security-review/checkers/fixtures/doc-drift/drift-stale-repo/worker-service/feature_1.py @@ -1 +1,2 @@ -def feature_1(): pass +def feature_1(): + pass diff --git a/security-review/checkers/fixtures/doc-drift/drift-stale-repo/worker-service/feature_2.py b/security-review/checkers/fixtures/doc-drift/drift-stale-repo/worker-service/feature_2.py index 8fa23b3..0bfe6d2 100644 --- a/security-review/checkers/fixtures/doc-drift/drift-stale-repo/worker-service/feature_2.py +++ b/security-review/checkers/fixtures/doc-drift/drift-stale-repo/worker-service/feature_2.py @@ -1 +1,2 @@ -def feature_2(): pass +def feature_2(): + pass diff --git a/security-review/checkers/fixtures/doc-drift/drift-stale-repo/worker-service/feature_3.py b/security-review/checkers/fixtures/doc-drift/drift-stale-repo/worker-service/feature_3.py index 34a52c1..2d396f7 100644 --- a/security-review/checkers/fixtures/doc-drift/drift-stale-repo/worker-service/feature_3.py +++ b/security-review/checkers/fixtures/doc-drift/drift-stale-repo/worker-service/feature_3.py @@ -1 +1,2 @@ -def feature_3(): pass +def feature_3(): + pass diff --git a/security-review/checkers/fixtures/doc-drift/drift-stale-repo/worker-service/feature_4.py b/security-review/checkers/fixtures/doc-drift/drift-stale-repo/worker-service/feature_4.py index 0c30159..1a21641 100644 --- a/security-review/checkers/fixtures/doc-drift/drift-stale-repo/worker-service/feature_4.py +++ b/security-review/checkers/fixtures/doc-drift/drift-stale-repo/worker-service/feature_4.py @@ -1 +1,2 @@ -def feature_4(): pass +def feature_4(): + pass diff --git a/security-review/checkers/fixtures/doc-drift/drift-stale-repo/worker-service/feature_5.py b/security-review/checkers/fixtures/doc-drift/drift-stale-repo/worker-service/feature_5.py index 062858f..40cc280 100644 --- a/security-review/checkers/fixtures/doc-drift/drift-stale-repo/worker-service/feature_5.py +++ b/security-review/checkers/fixtures/doc-drift/drift-stale-repo/worker-service/feature_5.py @@ -1 +1,2 @@ -def feature_5(): pass +def feature_5(): + pass diff --git a/security-review/checkers/fixtures/doc-drift/drift-stale-repo/worker-service/main.py b/security-review/checkers/fixtures/doc-drift/drift-stale-repo/worker-service/main.py index 4c21757..7424232 100644 --- a/security-review/checkers/fixtures/doc-drift/drift-stale-repo/worker-service/main.py +++ b/security-review/checkers/fixtures/doc-drift/drift-stale-repo/worker-service/main.py @@ -1 +1,2 @@ -class Worker: pass +class Worker: + pass diff --git a/security-review/checkers/fixtures/doc-drift/no-readme-repo/some-service/main.py b/security-review/checkers/fixtures/doc-drift/no-readme-repo/some-service/main.py index 2a50870..9985397 100644 --- a/security-review/checkers/fixtures/doc-drift/no-readme-repo/some-service/main.py +++ b/security-review/checkers/fixtures/doc-drift/no-readme-repo/some-service/main.py @@ -1 +1,2 @@ -class S: pass +class S: + pass