Page MenuHomeSoftware Heritage

apply re-formatting suggested by the black code formatter
ClosedPublic

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

Diff Detail

Repository
rDLDCVS CVS Loader
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 24178
Build 37740: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 37739: arc lint + arc unit

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.Oct 4 2021, 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.Oct 4 2021, 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.