Differential D6585

in cvs loader tests, use f-strings to build repository URLs
ClosedPublic
Actions

Authored by stsp on Oct 28 2021, 7:05 PM.

Details

Reviewers

vlorentz

Group Reviewers

Reviewers

Commits

rDLDCVSf36332c7e376: in cvs loader tests, use f-strings to build repository URLs

Summary

Suggested by ardumont in D6566

Diff Detail

Repository

rDLDCVS CVS Loader

Branch

master

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 24814
Build 38745: Phabricator diff pipeline on jenkins	Jenkins console · Jenkins
Build 38744: arc lint + arc unit

Event Timeline

stsp created this revision.Oct 28 2021, 7:05 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptOct 28 2021, 7:05 PM

stsp mentioned this in D6566: test checkout of file lacking trailing \n over pserver protocol.Oct 28 2021, 7:06 PM

Build is green

Patch application report for D6585 (id=23938)

Rebasing onto beb7fc8a02...

Current branch diff-target is up to date.

Changes applied before test

commit 2563ac0917db60491fd3d3218a47d16c97083d13
Author: Stefan Sperling <stsp@stsp.name>
Date:   Thu Oct 28 19:04:05 2021 +0200

    in cvs loader tests, use f-strings to build repository URLs
    
    Suggested by ardumont in D6566

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

Harbormaster completed remote builds in B24814: Diff 23938.Oct 28 2021, 7:06 PM

stsp requested review of this revision.Oct 28 2021, 7:06 PM

vlorentz accepted this revision.Nov 3 2021, 3:24 PM

This revision is now accepted and ready to land.Nov 3 2021, 3:24 PM

Closed by commit rDLDCVSf36332c7e376: in cvs loader tests, use f-strings to build repository URLs (authored by stsp). · Explain WhyDec 4 2021, 5:37 PM

This revision was automatically updated to reflect the committed changes.

stsp added a commit: rDLDCVSf36332c7e376: in cvs loader tests, use f-strings to build repository URLs.

Revision Contents
Changeset List

Path

Size

Coverage (All)

Coverage (Touched)

swh/

loader/

cvs/

tests/

test_loader.py

4 lines

100%

Diff 23938

View Options

swh/loader/cvs/tests/test_loader.py

in cvs loader tests, use f-strings to build repository URLsClosedPublicActions