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

Switch python toolchain to use ruff and uv #2404

Open
rachitnigam opened this issue Feb 2, 2025 · 2 comments
Open

Switch python toolchain to use ruff and uv #2404

rachitnigam opened this issue Feb 2, 2025 · 2 comments

Comments

@rachitnigam
Copy link
Contributor

ruff and uv are two new tools to make the formatting/linting and tool installation for python better. Let's switch from using the weird mix of flit, black, and flake8 to simply and consistently using ruff and uv (I know @sampsyo has already made this switch for Bril).

@EclecticGriffin
Copy link
Collaborator

If we want to do this migration in stages, the formatting side should be pretty simple. We can basically just run ruff format --check instead of the call to black. The linting is ruff check (though it seems there are a fair number of lints to fix).

Both can be accomplished via https://github.com/astral-sh/ruff-action.

@rachitnigam
Copy link
Contributor Author

Let's do it!

EclecticGriffin added a commit that referenced this issue Feb 27, 2025
Half of #2404.

The whole repo now has the python code format checked by `ruff format
--check` and linted by `ruff check`. _All_ python code is now subject to
these checks instead of being opt in.
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

No branches or pull requests

2 participants