Page MenuHomeSoftware Heritage

Lister.instance_name doesn't need a factory/default value
ClosedPublic

Authored by olasd on Tue, Jun 16, 10:59 AM.

Details

Summary

Depends on D3286. Got missed when doing D3271.

Diff Detail

Repository
rDSCH Scheduling utilities
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

olasd created this revision.Tue, Jun 16, 10:59 AM

Build is green

Patch application report for D3287 (id=11660)

Could not rebase; Attempt merge onto 1c93e553a1...

Updating 1c93e55..04894bd
Fast-forward
 swh/scheduler/model.py            | 51 +++++++++++++++++++++++++++++++++------
 swh/scheduler/tests/test_model.py | 19 ++++++++++++++-
 2 files changed, 61 insertions(+), 9 deletions(-)
Changes applied before test
commit 04894bd7fb6a1c4d658587395cbbe4f2d60c2a2a
Author: Nicolas Dandrimont <nicolas@dandrimont.eu>
Date:   Tue Jun 16 10:22:23 2020 +0200

    Lister.instance_name doesn't need a factory/default value

commit f520108a8d0abefec3a91967aedbc29fb1a808f8
Author: Nicolas Dandrimont <nicolas@dandrimont.eu>
Date:   Tue Jun 16 10:08:59 2020 +0200

    Improve support of primary keys
    
    This splits primary keys across "automatic" primary keys (handled by the
    database) and manual primary keys (managed by the user). Use the opportunity to
    improve/clarify the documentation of field metadata attributes.

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

ardumont accepted this revision.Tue, Jun 16, 2:39 PM
This revision is now accepted and ready to land.Tue, Jun 16, 2:39 PM