Skip to content

add ruff linting#31

Merged
lindestad merged 2 commits into
mainfrom
ruff-lint
Apr 29, 2026
Merged

add ruff linting#31
lindestad merged 2 commits into
mainfrom
ruff-lint

Conversation

@lindestad
Copy link
Copy Markdown
Collaborator

Summary

This is stacked on top of #30 and should be reviewed after the uv migration shape is agreed.

  • adds Ruff as a uv lint dependency group
  • configures Ruff formatting and a conservative lint rule set
  • adds a Checks workflow that runs Ruff on pull requests and pushes to main
  • applies Ruff formatting and the small lint fixes required by the new gate

Stack

Base PR: #30, uv-migration -> main

This PR: ruff-lint -> uv-migration

Once #30 lands, this PR can be retargeted to main or rebased onto main if GitHub does not update it cleanly.

Validation

  • uv run --frozen --group lint ruff format --check .
  • uv run --frozen --group lint ruff check .
  • uv run --frozen --group dev pytest -> 3 passed

@lindestad lindestad mentioned this pull request Apr 29, 2026
@lindestad lindestad requested a review from Gutteerz April 29, 2026 13:17
@lindestad lindestad marked this pull request as ready for review April 29, 2026 13:17
@lindestad lindestad changed the base branch from uv-migration to main April 29, 2026 14:23
@lindestad lindestad merged commit 7370fc7 into main Apr 29, 2026
1 check 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.

2 participants