Page MenuHomeSoftware Heritage

cli: Make cli work with new lister...
ClosedPublic

Authored by ardumont on Jan 27 2021, 6:38 PM.

Details

Summary

... while allowing legacy lister to still run (with --legacy)

Diff Detail

Event Timeline

Build is green

Patch application report for D4958 (id=17683)

Could not rebase; Attempt merge onto cbd2cce339...

Updating cbd2cce..b5160b8
Fast-forward
 swh/lister/cli.py                      |  8 +++++---
 swh/lister/gitlab/lister.py            |  6 +++++-
 swh/lister/gitlab/tests/test_lister.py | 10 ++++++++++
 3 files changed, 20 insertions(+), 4 deletions(-)
Changes applied before test
commit b5160b8a3692b271d3c6583bea4d855aa70cc880
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Wed Jan 27 18:28:09 2021 +0100

    cli: Make cli work with new lister
    
    while allowing legacy lister to still run (with --legacy)

commit 2065fb167d07b072c87d739c23f9f2c645d97fd0
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Wed Jan 27 18:02:08 2021 +0100

    gitlab: Allow sysadm to forget the trailing /

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

This revision is now accepted and ready to land.Jan 27 2021, 6:46 PM

Legacy option should be removed soon so no tests needed here ;-)

Rebase

(for the test and the soon to be adapted, yes, my thoughts exactly ;)

Build has FAILED

Patch application report for D4958 (id=17686)

Could not rebase; Attempt merge onto cbd2cce339...

Updating cbd2cce..40ef654
Fast-forward
 swh/lister/cli.py                      |  8 +++++---
 swh/lister/gitlab/lister.py            |  7 +++++--
 swh/lister/gitlab/tests/test_lister.py | 10 ++++++++++
 3 files changed, 20 insertions(+), 5 deletions(-)
Changes applied before test
commit 40ef654f6d7b8b079072d8af83a3593d9444edd3
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Wed Jan 27 18:28:09 2021 +0100

    cli: Make cli work with new lister
    
    while allowing legacy lister to still run (with --legacy)

commit 260c825953f6b5fe483d1cd408f86effe55ab71d
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Wed Jan 27 18:02:08 2021 +0100

    gitlab: Deal with missing or trailing / in url input

Link to build: https://jenkins.softwareheritage.org/job/DLS/job/tests-on-diff/206/
See console output for more information: https://jenkins.softwareheritage.org/job/DLS/job/tests-on-diff/206/console

Build is green

Patch application report for D4958 (id=17687)

Rebasing onto cbd2cce339...

Current branch diff-target is up to date.
Changes applied before test
commit 17b0e7af260c942093ac5d0c49e5f13ea7d9ed96
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Wed Jan 27 18:28:09 2021 +0100

    cli: Make cli work with new lister
    
    while allowing legacy lister to still run (with --legacy)

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

This revision was automatically updated to reflect the committed changes.