Differential D6399

apply re-formatting suggested by the black code formatter
ClosedPublic
Actions

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

Details

Reviewers

ardumont

Group Reviewers

Reviewers

Commits

rDLDCVS7f761b855071: apply re-formatting suggested by the black code formatter

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

stsp created this revision.Oct 4 2021, 12:52 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptOct 4 2021, 12:52 PM

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.

Harbormaster completed remote builds in B24178: Diff 23261.Oct 4 2021, 12:54 PM

stsp requested review of this revision.Oct 4 2021, 12:54 PM

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

In D6399#166050, @ardumont wrote:

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.

Closed by commit rDLDCVS7f761b855071: apply re-formatting suggested by the black code formatter (authored by stsp). · Explain WhyOct 4 2021, 2:03 PM

This revision was automatically updated to reflect the committed changes.

stsp added a commit: rDLDCVS7f761b855071: apply re-formatting suggested by the black code formatter.