search

aces / Loris-MRI

The set of scripts that preprocess and insert MRI data into the database.
10 stars 51 forks source link

Small linting improvements #1167

Closed maximemulder closed 2 months ago

maximemulder commented 3 months ago

While looking at linting for the LORIS-MRI automated testing initiative, I noticed that flake8 currently ignores rule E114, which states that tabulations must be 4 spaces. We obviously want this rule, so I un-ignored it and fixed the code that was not compliant.

There are also a few whitespace changes I missed in my last linting PR (didn't notice, my fault !). It should be good now.

maximemulder commented 2 months ago

@cmadjar There was a tabulation in a comment that I just removed, was it the one you were talking about ? I don't see any other one in our code.

image

EDIT : Damn, big screenshot is big.