Page MenuHomeSoftware Heritage

core.loader: Allow core loader to update origin_visit in one call
ClosedPublic

Authored by ardumont on Mar 26 2020, 3:30 PM.

Details

Summary

Prior to this commit, subclasses were in charge of setting the snapshot (for an
origin_visit).

There ended up being 2 origin_visit_update calls, 1 in the subclass to set
the snapshot id, then 1 in the loader core to set the visit (e.g. svn loader)

Motivation:
Make the origin_visit_update status parameter mandatory.

Preparatory work for T2310

Test Plan

tox

Diff Detail

Repository
rDLDBASE Generic VCS/Package Loader
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 11360
Build 17195: tox-on-jenkinsJenkins
Build 17194: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Mar 26 2020, 3:32 PM