Switch to using the black formatter

quantumlib / OpenFermion

The electronic structure package for quantum computers.

Apache License 2.0

1.51k stars 372 forks source link

Switch to using the black formatter #835

Closed mpharrigan closed 11 months ago

mpharrigan commented 11 months ago

copy check/format-incremental from Cirq (one minor edit to remove an unnecessary exclude line)
copy black version pin from Cirq so our open-source dev environments are mildly in sync.

Instead of formatting everything all at once, I plan on just letting the formatting change slowly over time as PRs format only their changed files. cc @ncrubin @fdmalone @tanujkhattar

fdmalone commented 11 months ago

Why not format everything in one go @mpharrigan? It seems like formatting incrementally will increase noise during review.

fdmalone commented 11 months ago

cirq did this all in one go right, with some git-blame-ignore-revs?

tanujkhattar commented 11 months ago

cirq did this all in one go right, with some git-blame-ignore-revs?

Yes

fdmalone commented 11 months ago

With that being said, I'd rather switch to black in one way or another. Can follow up with a bulk reformat.

fdmalone commented 11 months ago

xref #835