Skip to content

fix: start addressing ruff check --fix . issues#9

Merged
bassosimone merged 1 commit intomainfrom
refactor/lint
Nov 10, 2025
Merged

fix: start addressing ruff check --fix . issues#9
bassosimone merged 1 commit intomainfrom
refactor/lint

Conversation

@bassosimone
Copy link
Collaborator

This pull request starts addressing the issues surfacing when running:

uv run ruff check --fix .

I noticed that the current ruff configuration erred on the side of DRY but I would rather avoid that for domain clarity, so I disabled it.

In a subsequent diff, I'll address more issues that I preferred not to address now, lest we produce a large diff.

This pull request starts addressing the issues surfacing when running:

```bash
uv run ruff check --fix .
```

I noticed that the current `ruff` configuration erred on the side of DRY
but I would rather avoid that for domain clarity, so I disabled it.

In a subsequent diff, I'll address more issues that I preferred not to
address now, lest we produce a large diff.
@bassosimone bassosimone merged commit 2ee201e into main Nov 10, 2025
1 check passed
@bassosimone bassosimone deleted the refactor/lint branch November 10, 2025 18:29
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