Skip to content

tox.ini: Replace flake8 with ruff#184

Merged
corona10 merged 1 commit intopsf:mainfrom
cclauss:ruff
Apr 29, 2024
Merged

tox.ini: Replace flake8 with ruff#184
corona10 merged 1 commit intopsf:mainfrom
cclauss:ruff

Conversation

@cclauss
Copy link
Copy Markdown
Contributor

@cclauss cclauss commented Apr 29, 2024

https://docs.astral.sh/ruff

As recommended at #183 (comment)

Should we rename the tox job from pep8 to lint or not?

Copy link
Copy Markdown
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@corona10: What do you think?

@corona10 corona10 merged commit fdfd482 into psf:main Apr 29, 2024
@cclauss cclauss deleted the ruff branch April 29, 2024 18:38
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.

3 participants