svn.loader: Fix revision parents field to a tuple
AbandonedPublic
Actions

Authored by ardumont on Jun 4 2020, 12:08 PM.

Details

Reviewers

douardda
anlambert

Group Reviewers

Reviewers

Summary

Changing only the converters because that loader is currently mostly not typed... (so
modifying and adding type along sounds like taking too much time than needed
right now).

Also manipulating list within the loader sounds simpler than tuple. So taking
the lazy path here.

Related to D3177

Test Plan

tox

Diff Detail

Repository

rDLDSVN Subversion (SVN) loader

Branch

master

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 12627
Build 19186: Phabricator diff pipeline on jenkins	Jenkins console · Jenkins
Build 19185: arc lint + arc unit

Event Timeline

ardumont created this revision.Jun 4 2020, 12:08 PM

Build is green

Patch application report for D3219 (id=11423)

Rebasing onto eaf7a927c8...

Current branch diff-target is up to date.

Changes applied before test

commit d1b2f7b7b865fb833753e99ee91d1ff817c3fe98
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Thu Jun 4 12:05:23 2020 +0200

    svn.loader: Fix revision parents field to a tuple
    
    Related to D3177

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

Harbormaster completed remote builds in B12627: Diff 11423.Jun 4 2020, 12:10 PM

ardumont edited the summary of this revision. (Show Details)Jun 4 2020, 12:10 PM

ardumont mentioned this in D3222: Adapt to swh.model 0.3.Jun 4 2020, 3:25 PM

ardumont abandoned this revision.Jun 4 2020, 3:48 PM