ComPWA / gluex-nstar

Investigation of N* in GlueX photoproduction experiment
https://compwa.github.io/gluex-nstar
MIT License
2 stars 1 forks source link

Add Ruff configuration #40

Closed redeboer closed 3 months ago

redeboer commented 4 months ago

Ruff is currently using the default configuration, but there are many additional rules that can are quite useful. (For the open PRs, we need ERA001 😁 ).

See for example which rules AmpForm selects. But that's a bit too much, for now just activate ERA001

shenvitor commented 4 months ago

ERA001 Man... that's commented-out-code 😬

shenvitor commented 3 months ago

UP034 To be added too

redeboer commented 3 months ago

I would even select ALL. Since Ruff v0.5.0, it's quite stable to use that. Any rules we do not like can then be deactivated.