model: Add OriginVisitUpdate model object (!272) · Merge requests · Platform / Development / swh-model · GitLab

Snippets Groups Projects

Closed Antoine R. Dumont requested to merge generated-differential-D2880-source into generated-differential-D2880-target 5 years ago

1 unresolved thread

(pairing with @vlorentz)

Need to be updated according to @douardda's latest improvments on model (if that got merged before ;)

Related to T2310

Test Plan

tox

Migrated from D2880 (view on Phabricator)

Activity

Software Heritage Continuous Integration for public repositories @swh-public-ci · 5 years ago

Build is green See https://jenkins.softwareheritage.org/job/DMOD/job/tox/224/ for more details.
Antoine R. Dumont @ardumont · 5 years ago

Author Maintainer

Add tests

Software Heritage Continuous Integration for public repositories @swh-public-ci · 5 years ago

Build is green

Patch application report for D2880 (id=10431)

Rebasing onto accca603...

Current branch diff-target is up to date.

Changes applied before test

commit f35783e3e9e01f6c80af75ec8ee97376fa9faabf
Author: Antoine R. Dumont (@ardumont) <antoine.romain.dumont@gmail.com>
Date:   Wed Mar 25 18:00:03 2020 +0100

    model: Add new OriginVisitUpdate model object + test strategy
    
    (pairing with @vlorentz)
    
    Related to [T2310](https://forge.softwareheritage.org/T2310 'view original for T2310 on Phabricator')

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

vlorentz @vlorentz started a thread on the diff 5 years ago

swh/model/hypothesis_strategies.py

         origin=urls(),
         status=sampled_from(['ongoing', 'full', 'partial']),
         type=pgsql_text(),
         snapshot=optional(sha1_git()))
         snapshot=optional(sha1_git()),
+    )
 def metadata_dicts():

vlorentz @vlorentz · 5 years ago

Maintainer

lgtm, but I'll let @douardda accept the diff

Phabricator Migration user @phabricator-migration commented on commit e9a4c751 5 years ago

Owner
Some references in the commit message have been migrated:

T2310 is now swh/meta#2310 (closed)

Antoine R. Dumont @ardumont · 5 years ago

Author Maintainer

Adapt according to review

Software Heritage Continuous Integration for public repositories @swh-public-ci · 5 years ago

Build is green

Patch application report for D2880 (id=10433)

Rebasing onto accca603...

Current branch diff-target is up to date.

Changes applied before test

commit e9a4c7519e4e2d9763294461f8628a519a3af308
Author: Antoine R. Dumont (@ardumont) <antoine.romain.dumont@gmail.com>
Date:   Wed Mar 25 18:00:03 2020 +0100

    model: Add new OriginVisitUpdate model object + test strategy
    
    (pairing with @vlorentz)
    
    Related to [T2310](https://forge.softwareheritage.org/T2310 'view original for T2310 on Phabricator')

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

David Douard mentioned in merge request swh-storage!918 (closed) 5 years ago

mentioned in merge request swh-storage!918 (closed)

David Douard @douardda · 5 years ago

Maintainer

Merge request was accepted

David Douard approved this merge request 5 years ago

approved this merge request

Antoine R. Dumont @ardumont · 5 years ago

Author Maintainer

Merge request was merged

Antoine R. Dumont closed 5 years ago

closed

Phabricator Migration user mentioned in issue swh/meta#2310 (closed) 2 years ago

mentioned in issue swh/meta#2310 (closed)

Please register or sign in to reply