Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update default setup for linting and reformatting Darker itself #552

Merged
merged 7 commits into from
Mar 11, 2024

Conversation

akaihola
Copy link
Owner

@akaihola akaihola commented Mar 9, 2024

No description provided.

@akaihola akaihola added the CI label Mar 9, 2024
@akaihola akaihola added this to the Darker 2.0.0 - Drop py37 milestone Mar 9, 2024
@akaihola akaihola self-assigned this Mar 9, 2024
@akaihola akaihola force-pushed the darker-config-ruff-update branch 3 times, most recently from 9ca77d7 to ef4e38b Compare March 11, 2024 17:50
By default, the command `darker` now runs Black on the source code of
Darker, the Darker GitHub Action, release tooling and `setup.py`.

`--config=check-darker.toml` can now be used to run Flake8, Mypy,
Pydocstyle, Pylint and Ruff on modified lines in above mentioned files.
Those tools are included when installing with the `[test]` extra.

The minimum ruff version is now 0.0.292, and its configuration in
`pyproject.toml` has been updated accordingly.
It's a bit outdated. We now run Flake8 in its own workflow, and check
formatting using the Darker GitHub Action in another workflow.
@akaihola akaihola force-pushed the darker-config-ruff-update branch from ef4e38b to 1644e32 Compare March 11, 2024 18:07
@akaihola akaihola force-pushed the darker-config-ruff-update branch from 1644e32 to 7a6c6cc Compare March 11, 2024 20:36
@akaihola akaihola merged commit a13af46 into master Mar 11, 2024
35 of 37 checks passed
@akaihola akaihola deleted the darker-config-ruff-update branch March 11, 2024 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

Successfully merging this pull request may close these issues.

1 participant