Page MenuHomeSoftware Heritage

Save code now: Do not check origins for existence
ClosedPublic

Authored by ardumont on May 3 2021, 2:46 PM.

Details

Summary

This check will only work for existing origin visit types (svn, hg, git) which are
behind a frontend.

This will be re-opened soon but restricted to the new incoming bundle visit type (for
the associated artifacts).

Related to T3213

Test Plan

tox

Diff Detail

Repository
rDWAPPS Web applications
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 21212
Build 32931: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 32930: arc lint + arc unit

Event Timeline

Drop some more missed assertions the first time around

Build is green

Patch application report for D5661 (id=20221)

Rebasing onto 8439eb5c9b...

Current branch diff-target is up to date.
Changes applied before test
commit f4c36fb5d4d63b953d738c9fe9cf93532b2b9d69
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Mon May 3 14:23:52 2021 +0200

    Save code now: Do not check origins for existence
    
    This check will only work for existing origin visit types (svn, hg, git) which are
    behind a frontend.
    
    This will be re-opened soon but restricted to the new incoming bundle visit type (for
    the associated artifacts).
    
    Related to T3213

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

Build is green

Patch application report for D5661 (id=20222)

Rebasing onto 8439eb5c9b...

Current branch diff-target is up to date.
Changes applied before test
commit 36740a0fd9b03d08772ccd1ec50c8d3690c6978f
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Mon May 3 14:23:52 2021 +0200

    Save code now: Do not check origins for existence
    
    This check will only work for existing origin visit types (svn, hg, git) which are
    behind a frontend.
    
    This will be re-opened soon but restricted to the new incoming bundle visit type (for
    the associated artifacts).
    
    Related to T3213

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

This revision is now accepted and ready to land.May 3 2021, 3:08 PM