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.

Sign up for GitHub

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

Jump to bottom

Switch from flake8/yapf to ruff #1702

Open
joaander opened this issue Jan 22, 2024 · 0 comments
Open

Switch from flake8/yapf to ruff #1702

joaander opened this issue Jan 22, 2024 · 0 comments
Labels
breaking Changes that will break API. task Something needs to be done.

Comments

@joaander
Copy link
Member

Description

Change the pre-commit configuration from flake8/yapf to ruff.

Due to the massive number of changes, this should be done as the final change on trunk-major just prior to releasing 5.0.

Motivation and context

ruff performs more linting checks built in than flake8 and formats much faster than yapf.
@joaander joaander added breaking Changes that will break API. task Something needs to be done. labels Jan 22, 2024
@joaander joaander changed the title Switch from rlake8/yapf to ruff Switch from flake8/yapf to ruff Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
breaking Changes that will break API. task Something needs to be done.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@joaander