Create pyproject.toml to define linter rules #6950

Closed
opened 2025-12-29 19:47:07 +01:00 by adam · 1 comment
Owner

Originally created by @jeremystretch on GitHub (Sep 8, 2022).

Originally assigned to: @jsenecal on GitHub.

Proposed Changes

A proposal was raised during today's maintainers' meeting to introduce a pyproject.toml (see PEP 518) file to define the linting rules used in the project. We currently have a pre-commit hook which (among other things) runs pycodestyle with a few arguments to control the rules being applied. The goal here is to document these rules for discovery by a contributor's IDE.

Justification

Lowers development friction for contributors using different IDEs and tool sets.

Originally created by @jeremystretch on GitHub (Sep 8, 2022). Originally assigned to: @jsenecal on GitHub. ### Proposed Changes A proposal was raised during today's maintainers' meeting to introduce a `pyproject.toml` (see [PEP 518](https://peps.python.org/pep-0518/)) file to define the linting rules used in the project. We currently have a pre-commit hook which (among other things) runs `pycodestyle` with a few arguments to control the rules being applied. The goal here is to document these rules for discovery by a contributor's IDE. ### Justification Lowers development friction for contributors using different IDEs and tool sets.
adam added the status: acceptedtype: housekeeping labels 2025-12-29 19:47:07 +01:00
adam closed this issue 2025-12-29 19:47:07 +01:00
Author
Owner

@jsenecal commented on GitHub (Sep 9, 2022):

Turns out that pycodestyle rejected the idea of supporting pyproject.toml file for configuration as noted in this pinned issue (Thanks Jeremy)

This may end up bigger than expected if we have to move to something else?

@jsenecal commented on GitHub (Sep 9, 2022): Turns out that pycodestyle rejected the idea of supporting pyproject.toml file for configuration as noted in [this pinned issue](https://github.com/PyCQA/pycodestyle/issues/813) (Thanks Jeremy) This may end up bigger than expected if we have to move to something else?
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#6950