Skip to content

strengthen Ruff linting and formatting - #28

Merged
laceyp99 merged 2 commits into
mainfrom
t3code/upgrade-ruff-lint-rules
Aug 14, 2026
Merged

laceyp99 merged 2 commits into
mainfrom
t3code/upgrade-ruff-lint-rules

Conversation

@laceyp99

Copy link
Copy Markdown
Owner

Summary

  • upgrade the development toolchain to Ruff 0.16.2 and enable a broader curated set of correctness, modernization, testing, path, performance, and Ruff-native checks
  • adopt Ruff's 88-character formatting baseline across evaluation orchestration, deterministic checks, analysis, and tests
  • resolve the new lint findings with safer defaults, timezone-aware timestamps, explicit concurrency and iteration behavior, modern path handling, and focused test updates

Testing

  • uv sync --locked --all-extras --python 3.12 - passed
  • uv run --locked --all-extras --python 3.12 ruff format --check . - passed
  • uv run --locked --all-extras --python 3.12 ruff check . - passed
  • uv run --locked --all-extras --python 3.12 pytest -q - passed (130 tests)
  • uv build - passed (source distribution and wheel)

@laceyp99
laceyp99 marked this pull request as ready for review August 14, 2026 04:34

@laceyp99 laceyp99 left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me! This is a cross project effort to make line length formatting uniform and have project relevant lint rules. This seems like a ton of LOC changes, but its mainly two simple config changes.

@laceyp99
laceyp99 merged commit 62cdf90 into main Aug 14, 2026
2 checks passed
@laceyp99
laceyp99 deleted the t3code/upgrade-ruff-lint-rules branch August 14, 2026 04:45
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