Skip to content

Apply ruff format autoformatter to src/ + tests/ (ar-r82f.17)#97

Open
atc964 wants to merge 1 commit into
mainfrom
fix/ruff-format-sweep
Open

Apply ruff format autoformatter to src/ + tests/ (ar-r82f.17)#97
atc964 wants to merge 1 commit into
mainfrom
fix/ruff-format-sweep

Conversation

@atc964
Copy link
Copy Markdown
Collaborator

@atc964 atc964 commented May 28, 2026

Summary

Apply ruff format to 80 files in src/ and tests/. This is the autoformatter complement to PR #95's ruff check cleanup (which Lila brought from 591 → 0 errors).

Pure formatting changes only: whitespace, quote style, line breaks, indentation. Zero behavior change — deterministic autoformat.

Verification

  • Before: ruff format --check src/ tests/ → 80 files would be reformatted
  • After: ruff format --check src/ tests/ → All checks passed (0 violations)
  • ruff check src/ tests/ → All checks passed (PR Lint cleanup sweep: 591 -> 95 errors (ar-r82f.14) #95 cleanup preserved)
  • Test suite: 3225 passed, 4 skipped (no regressions vs main)
  • No source logic touched; no pyproject.toml, workflows, or non-Python files modified

Why now

With this in, the Lint job should be green for the first time in months (both ruff check and ruff format --check pass).

Test plan

  • ruff format --check src/ tests/ exits 0
  • ruff check src/ tests/ exits 0
  • Full unit + integration test suite passes locally
  • CI: Lint job PASS
  • CI: Test job PASS
  • CI: Docker Build PASS

bead: ar-r82f.17

Co-Authored-By: Claude Sonnet 4.5 noreply@anthropic.com

Apply the `ruff format` autoformatter to 80 files (whitespace, quote
style, line breaks, indentation). Complement to PR #95's `ruff check`
cleanup. Zero behavior change — deterministic autoformat only.

After:
- `ruff format --check src/ tests/` → All checks passed
- `ruff check src/ tests/` → All checks passed (PR #95 cleanup preserved)
- Test suite: 3225 passed, 4 skipped (no regressions)

bead: ar-r82f.17

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant