Differential D4405

Drop todo from the specification
ClosedPublic
Actions

Authored by ardumont on Nov 3 2020, 3:39 PM.

Details

Reviewers

moranegg

Group Reviewers

Reviewers

Maniphest Tasks

T2625: create and publish xml schema for the specific swh-deposit metadata

Commits

rDDEPd8d4e0bc9a01: Drop todo from the specification

Summary

It's actually in the task referenced here, no need to clutter the spec ;)

Related to T2625

Test Plan

tox

Diff Detail

Repository

rDDEP Push deposit

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Event Timeline

ardumont created this revision.Nov 3 2020, 3:39 PM

ardumont mentioned this in D4403: Update metadata-only deposit specification.Nov 3 2020, 3:39 PM

Build is green

Patch application report for D4405 (id=15603)

Rebasing onto be5fd4afe0...

Current branch diff-target is up to date.

Changes applied before test

commit d8d4e0bc9a01d9439739559ea30609946c2b998e
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Tue Nov 3 15:39:36 2020 +0100

    Drop todo from the specification
    
    Related to T2625

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

Harbormaster completed remote builds in B16826: Diff 15603.Nov 3 2020, 3:42 PM

ardumont mentioned this in T2537: Extend new deposit endpoint to support metadata-only deposits.Nov 3 2020, 4:02 PM

moranegg added inline comments.Nov 3 2020, 4:43 PM

docs/specs/spec-meta-deposit.rst
38	I want to accept this diff, but I can't before knowing that this example is correct, which is not the case at the moment.

ardumont added inline comments.Nov 3 2020, 4:49 PM

docs/specs/spec-meta-deposit.rst
38	That should be fixed alongside the part that needs fixing with the T2625 perimeter.

moranegg accepted this revision.Nov 3 2020, 5:02 PM

This revision is now accepted and ready to land.Nov 3 2020, 5:02 PM

Closed by commit rDDEPd8d4e0bc9a01: Drop todo from the specification (authored by ardumont). · Explain WhyNov 3 2020, 5:03 PM

This revision was automatically updated to reflect the committed changes.

ardumont added a commit: rDDEPd8d4e0bc9a01: Drop todo from the specification.

Revision Contents
Changeset List

Path

Size

docs/

specs/

spec-meta-deposit.rst

2 lines

Diff 15608

View Options

docs/specs/spec-meta-deposit.rst

Drop todo from the specificationClosedPublicActions