Skip to content

chore: apply ruff semantics-preserving autofixes - #291

Merged
eduardstan merged 1 commit into
mainfrom
fm/rs-gitroll-mech
Aug 20, 2026
Merged

chore: apply ruff semantics-preserving autofixes#291
eduardstan merged 1 commit into
mainfrom
fm/rs-gitroll-mech

Conversation

@eduardstan

Copy link
Copy Markdown
Owner

Evidence

$ ruff check .
All checks passed!

$ ruff check . --select 'UP045,UP037,UP035,UP017,UP033,RUF022,C420,PIE790,PIE808,RET501'
All checks passed!

$ mypy
Success: no issues found in 2 source files

$ pytest --cov=reasonsmith --cov-report=term-missing --cov-fail-under=95
1633 passed, 36 skipped, 3 warnings in 548.57s (0:09:08)
TOTAL 10673 521 95.12%
Required test coverage of 95% reached. Total coverage: 95.12%

$ git diff --stat main...HEAD
43 files changed, 168 insertions(+), 164 deletions(-)

The complete pytest output includes the per-file term-missing table and the pytest-asyncio deprecation warning; the command exited 0. The diff was read hunk-by-hunk: it changes no string literal, no control flow, and no public signature. Optional annotations becoming PEP 604 unions are expected annotation-only changes; defaults and exception behavior are unchanged.

packages/** TypeScript is untouched. The two example files were authorized by firstmate: src/reasonsmith/examples/language_model_notices.py (UP035 Callable import) and src/reasonsmith/examples/recounted_reason_trace.py (RET501). The CI-ruleset cleanup produced no files outside the 43 already touched by the selected autofixes; git diff --name-only before and after was unchanged.

RUF100 was intentionally excluded because it would remove suppressions that remain load-bearing under other rule sets.

@eduardstan
eduardstan merged commit caab71c into main Aug 20, 2026
11 checks passed
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