Adding black as linter?

stefanv · May 11, 2023, 6:40pm

If we can find any formatter that formats arrays better, I’d go with that. @jni, I think, prefers cercis, but it also formats arrays weirdly (as per the example in the astropy APE).

I don’t think we need a SKIP on this, just (a) a workable tool and (b) consensus at least among the core team.

lagru · October 14, 2023, 12:56pm

Fyi

github.com/scikit-image/scikit-image

Use Black

scikit-image:main ← jarrodmillman:black

opened 05:09PM - 10 Oct 23 UTC

jarrodmillman

+20741 -13566

## Description Fixes #7195. I use black for almost every other project I work… on and I have become used to its formatting and prefer to rely on it rather than spending my time thinking about formatting. The discussion on the developer's list seemed to be generally ok with using black with some minor concerns. I decided to just make the PR and see what happens. In e7f6abc510a419343df2a8abaca6de9ac5adfd9d, I told black to avoid reformatting some np arrays. I can remove that or redo it to include more. I just didn't want to spend too much time on this PR until I got some feedback from others.

lagru · October 18, 2023, 11:34am

See Apply black to `_hog.py` after previous merge lacking black by lagru · Pull Request #7215 · scikit-image/scikit-image · GitHub.

For a while we’ll probably want to make sure that we merge main (the black commit hook config) into the PR branch before merging.