Skip to content

Commit 3ee9970

Browse files
fix: resolve ruff I001 import sorting and E501 line-too-long violations
Fix ruff I001 import sorting violations across scanner.py, test_config_and_fixes.py, and test_scanner.py. All 4 errors resolved via ruff --fix. +22/-6 across 4 files. Closes #33
1 parent 0ccab80 commit 3ee9970

4 files changed

Lines changed: 22 additions & 6 deletions

File tree

src/deadcode/cli.py

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,13 @@ def _get_fail_threshold(ctx: click.Context) -> int:
7373
@click.option("--fail", "fail_threshold", type=int, default=None,
7474
help="Exit code 1 if findings >= threshold (overrides .deadcode.yml)")
7575
@click.pass_context
76-
def scan(ctx: click.Context, json_output: bool, format: str | None, category: str | None, fail_threshold: int | None) -> None:
76+
def scan(
77+
ctx: click.Context,
78+
json_output: bool,
79+
format: str | None,
80+
category: str | None,
81+
fail_threshold: int | None,
82+
) -> None:
7783
"""Scan project for dead code."""
7884
project = ctx.obj["project"]
7985
ignore = _merge_config_ignore(ctx)

src/deadcode/scanner.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,11 +10,12 @@
1010
from __future__ import annotations
1111

1212
import os
13-
import pathspec
1413
import re
1514
from dataclasses import dataclass, field
1615
from pathlib import Path
1716

17+
import pathspec
18+
1819
# ── Data structures ───────────────────────────────────────────────────
1920

2021

tests/test_config_and_fixes.py

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,9 @@
33
from __future__ import annotations
44

55
import json
6+
67
import pytest
8+
79
from deadcode.cli import cli
810
from deadcode.config import DeadCodeConfig
911
from deadcode.scanner import DeadCodeScanner
@@ -232,8 +234,9 @@ class TestLicenseDepRemoved:
232234

233235
def test_no_license_import_in_cli(self):
234236
"""The cli module should not reference revenueholdings_license."""
235-
import deadcode.cli as cli_mod
236237
import inspect
238+
239+
import deadcode.cli as cli_mod
237240
source = inspect.getsource(cli_mod)
238241
assert "revenueholdings_license" not in source
239242
assert "require_license" not in source
@@ -288,7 +291,9 @@ def test_include_multiple_dirs(self, runner, tmp_path):
288291
lib.parent.mkdir(parents=True, exist_ok=True)
289292
lib.write_text('export function bar() { return 2; }\n')
290293

291-
result = runner.invoke(cli, ["-p", str(tmp_path), "--include", "src/", "--include", "lib/", "scan", "--json-output"])
294+
result = runner.invoke(
295+
cli, ["-p", str(tmp_path), "--include", "src/", "--include", "lib/", "scan", "--json-output"],
296+
)
292297
assert result.exit_code == 0
293298
data = json.loads(result.output, strict=False)
294299
assert data["files_scanned"] == 2
@@ -331,7 +336,9 @@ def test_include_with_ignore_more_restrictive(self, runner, tmp_path):
331336
internal.parent.mkdir(parents=True, exist_ok=True)
332337
internal.write_text('export function bar() { return 2; }\n')
333338

334-
result = runner.invoke(cli, ["-p", str(tmp_path), "--include", "src/", "-i", "src/internal/", "scan", "--json-output"])
339+
result = runner.invoke(
340+
cli, ["-p", str(tmp_path), "--include", "src/", "-i", "src/internal/", "scan", "--json-output"],
341+
)
335342
assert result.exit_code == 0
336343
data = json.loads(result.output, strict=False)
337344
assert data["files_scanned"] == 1, "only mod.ts should be scanned, internal/ ignored"

tests/test_scanner.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,12 @@
33
from __future__ import annotations
44

55
import json
6+
from pathlib import Path
7+
68
import pytest
9+
710
from deadcode.cli import cli
811
from deadcode.scanner import DeadCodeScanner
9-
from pathlib import Path
1012

1113

1214
@pytest.fixture

0 commit comments

Comments
 (0)