tweag / FawltyDeps

Python dependency checker
Other
201 stars 14 forks source link

Add the ruff linter #424

Closed jherland closed 6 months ago

jherland commented 7 months ago

A minimal introduction of ruff. The first in a series of 3 PRs.

Commits:

jherland commented 6 months ago

Is this a poetry-focused usage of pyproject.toml which disallows that?

Exactly. Since we're using Poetry, we're currently defining our project metadata in the Poetry-specific [tool.poetry] section in our pyproject.toml.

[project]
requires-python = ">=3.8"

This is from the PEP621 standard for specifying project metadata, and this standard seems to have more momentum than Poetry's own format, so I'd like to migrate to this at some point. Either we see if Poetry can support the more standard format, or we look around for alternatives to Poetry. But that is a separate issue/effort than this PR. 😄

jherland commented 6 months ago

Either we see if Poetry can support the more standard format, or we look around for alternatives to Poetry. But that is a separate issue/effort than this PR.

Created https://github.com/tweag/FawltyDeps/issues/433 to keep discussing our future with/without Poetry