Differential D3691

nixguix: Fix docstring
ClosedPublic
Actions

Authored by ardumont on Aug 3 2020, 4:50 PM.

Details

Reviewers

vlorentz

Group Reviewers

Reviewers

Commits

rDLDBASE3d73cd0b4185: nixguix: Fix docstring

Summary

To have a better grasp of what this does.

Suggestion improvments more than welcome ;)

Test Plan

tox

Diff Detail

Repository

rDLDBASE Generic VCS/Package Loader

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Event Timeline

ardumont created this revision.Aug 3 2020, 4:50 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptAug 3 2020, 4:50 PM

ardumont edited the summary of this revision. (Show Details)Aug 3 2020, 4:52 PM

Build is green

Patch application report for D3691 (id=12998)

Could not rebase; Attempt merge onto 614329b7eb...

Updating 614329b..3d73cd0
Fast-forward
 swh/loader/package/nixguix/loader.py | 22 ++++++++++++++++------
 1 file changed, 16 insertions(+), 6 deletions(-)

Changes applied before test

commit 3d73cd0b41858487771f342b7a5a84a8bcd6fb67
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Mon Aug 3 16:42:41 2020 +0200

    nixguix: Fix docstring

commit 889d9b75f42340494b3a904fba544d0ba6e4726a
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Mon Aug 3 16:42:27 2020 +0200

    nixguix: Align error message formatting using f-string

commit da1c18a11ebaa0ea51375b11b10823c17888ce7b
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Mon Aug 3 16:42:14 2020 +0200

    nixguix: Fix format issue in error message

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

Harbormaster completed remote builds in B14239: Diff 12998.Aug 3 2020, 4:52 PM

vlorentz accepted this revision.Aug 3 2020, 5:57 PM

This revision is now accepted and ready to land.Aug 3 2020, 5:57 PM

Closed by commit rDLDBASE3d73cd0b4185: nixguix: Fix docstring (authored by ardumont). · Explain WhyAug 3 2020, 6:06 PM

This revision was automatically updated to reflect the committed changes.

ardumont added a commit: rDLDBASE3d73cd0b4185: nixguix: Fix docstring.