brainglobe-atlasapi icon indicating copy to clipboard operation
brainglobe-atlasapi copied to clipboard

Published 20 hours ago • verified publisher icon brainglobe

Adding pycln

Open FedeClaudi opened this issue 3 years ago • 4 comments

Hi,

Opening it here but valid throughout the org. We could add pycln to our pre-commit stack. It automatically removes unused imports from all files. I have set it up in pyinspect to run before flake8 so that it fixed is before flake8 stops the commit from going through.

Lemme know!

FedeClaudi avatar Oct 05 '20 16:10 FedeClaudi

cc @adamltyson @vigji

FedeClaudi avatar Oct 05 '20 16:10 FedeClaudi

sounds good to me!

vigji avatar Oct 06 '20 06:10 vigji

Yep sounds good.

adamltyson avatar Oct 06 '20 06:10 adamltyson

Ooops, I got a 404 error when I first tried to open the issue, it seems that it opened it twice instread! Sorry 'bout that

FedeClaudi avatar Oct 06 '20 10:10 FedeClaudi