Stricter linter configuration

aces / Loris-MRI

The set of scripts that preprocess and insert MRI data into the database.

10 stars 51 forks source link

Closed maximemulder closed 1 month ago

maximemulder commented 1 month ago

Adds more lints to our Ruff configuration.

Open questions:

Should we add "N818" ? (exception classes should be named FooError instead of Foo or FooException)
Should we add "I" lints (imports must be sorted) ? (autofixable with ruff check --fix)

maximemulder commented 1 month ago

I un-ignored N818 in adb290. I kept the directory name "exception" but if you want me to rename it to "error" or "errors" it's fine I can do it.

I added import sorting in 8abdce. The only changes I did manually are in pyproject.toml, the rest is just ruff check --fix.

PS: Some things are nice to see 😌