Page MenuHomeSoftware Heritage

apply re-formatting suggested by the black code formatter
ClosedPublic

Authored by stsp on Mon, Oct 4, 12:52 PM.

Diff Detail

Repository
rDLDCVS CVS Loader
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Build is green

Patch application report for D6399 (id=23261)

Rebasing onto ea469457ed...

Current branch diff-target is up to date.
Changes applied before test
commit 7f761b855071cd7512c51fe59930b78323da5720
Author: Stefan Sperling <stsp@stsp.name>
Date:   Mon Oct 4 12:51:24 2021 +0200

    apply re-formatting suggested by the black code formatter

See https://jenkins.softwareheritage.org/job/DLDCVS/job/tests-on-diff/32/ for more details.

stsp requested review of this revision.Mon, Oct 4, 12:54 PM
ardumont added a subscriber: ardumont.

Good idea.

Thanks.

Do you have the pre-commit installed in that repository yet? [1]

[1] https://docs.softwareheritage.org/devel/developer-setup.html#checkout-the-source-code

This revision is now accepted and ready to land.Mon, Oct 4, 1:05 PM

Do you have the pre-commit installed in that repository yet? [1]

[1] https://docs.softwareheritage.org/devel/developer-setup.html#checkout-the-source-code

Yes, I do. I have been ignoring black's warnings on this file for a while because I had no good solution for the scramble table becoming unreadable. This has been fixed with D6389 so running black on this is no longer an issue.