Differential D4518

Split _check_preconditions_on into _check_file_length and _check_file_md5sum.
ClosedPublic
Actions

Authored by vlorentz on Nov 19 2020, 10:37 AM.

Details

Reviewers

ardumont

Group Reviewers

Reviewers

Commits

rDDEP0e72920bb8bb: Split _check_preconditions_on into _check_file_length and _check_file_md5sum.

Summary

they are better function names IMO.

Diff Detail

Repository

rDDEP Push deposit

Lint

Automatic diff as part of commit; lint not applicable.

Unit

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

Event Timeline

vlorentz created this revision.Nov 19 2020, 10:37 AM

Herald added a reviewer: Reviewers. · View Herald TranscriptNov 19 2020, 10:37 AM

Build is green

Patch application report for D4518 (id=16019)

Could not rebase; Attempt merge onto bacbc29731...

Updating bacbc297..0e72920b
Fast-forward
 swh/deposit/api/common.py | 60 ++++++++++++++++++++++++++++-------------------
 1 file changed, 36 insertions(+), 24 deletions(-)

Changes applied before test

commit 0e72920bb8bba285abd2f201aa934a626824b681
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Nov 19 10:37:34 2020 +0100

    Split _check_preconditions_on into _check_file_length and _check_file_md5sum.
    
    they are better function names IMO.

commit 43edd9039f23a5ff449897ba7ff4c59ac3868197
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Wed Nov 18 16:31:09 2020 +0100

    Remove duplicate type annotations.
    
    They are already in the signature.

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

Harbormaster completed remote builds in B17226: Diff 16019.Nov 19 2020, 10:40 AM

ardumont added a subscriber: ardumont.Nov 19 2020, 11:59 AM

ardumont added inline comments.