Skip to content

Conversation

@Preocts
Copy link
Owner

@Preocts Preocts commented May 30, 2025

Move black, flake8, and isort actions into the noxfile.py alongside
the existing mypy session. This will give the dev control over when
these checks are run, allowing pre-commit to be optional in the local
environment.

Add

  • nox -s lint session to run formatters, linters, and type checking

Update

  • CI to use new lint session in nox

Remove

  • mypy nox session
  • black configuration section from pyproject.toml

Preocts added 3 commits May 30, 2025 02:41
Move black, flake8, and isort actions into the `noxfile.py` alongside
the existing mypy session. This will give the dev control over when
these checks are run, allowing pre-commit to be optional in the local
environment.

There is a speed drawback as pre-commit runs on edited files only and
nox will run on all file. The pre-commit config will remain and is
always an option locally and in pre-commit.ci.
@Preocts Preocts self-assigned this May 30, 2025
@Preocts Preocts marked this pull request as ready for review May 30, 2025 07:05
@Preocts Preocts enabled auto-merge (squash) May 30, 2025 07:05
@Preocts Preocts merged commit 3040992 into main May 30, 2025
18 checks passed
@Preocts Preocts deleted the even-more-nox-updates branch May 30, 2025 07:06
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