search

cctbx / dxtbx

Diffraction Experiment Toolbox
BSD 3-Clause "New" or "Revised" License
2 stars 18 forks source link

Switch linting/formatting to ruff #738

Closed ndevenish closed 3 months ago

ndevenish commented 4 months ago

Away from flake8/black. This is much faster, allows us to drop setup.cfg, gives us many more options for linting, and enables pre-commit to autofix many of the standard issues (only safe fixes are applied by default, however).

Our version of black in pre-commit was quite old, so a good chunk of these changes were from upgrading and running black first.

codecov[bot] commented 3 months ago

Codecov Report

Attention: Patch coverage is 22.72727% with 34 lines in your changes missing coverage. Please review.

Project coverage is 41.76%. Comparing base (09de2a8) to head (a261b39).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #738 +/- ## ======================================= Coverage 41.76% 41.76% ======================================= Files 188 188 Lines 16877 16877 Branches 3233 3233 ======================================= Hits 7048 7048 - Misses 9174 9179 +5 + Partials 655 650 -5 ```