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

Reinstate migration to the Ruff formatter #2311

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

whabanks
Copy link
Contributor

@whabanks whabanks commented Sep 27, 2024

Reverts #2289 reinstating the Ruff formatter addition to the codebase.

  • Updated Ruff to 0.8.2.
  • Updated Ruff VScode extension to prevent automatic removal of unused imports on save. Unused imports will still be removed when make format is ran.
  • Ruff will now properly check formatting in CI.

Incorporates additions from:

@whabanks whabanks changed the title Revert "Revert "Experiment - Add Ruff formatter/linter"" Reinstate migration to the Ruff formatter Sep 27, 2024
@whabanks whabanks requested a review from jzbahrai December 2, 2024 14:07
whabanks and others added 5 commits December 6, 2024 14:17
- The VSCode ruff extension will no longer automatically remove unused imports but will still notify you they're incorrect
- Updated ruff to 0.8.2
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