Page MenuHomeSoftware Heritage

spec-technical: Add 'partial' self-loop + annotate transitions in the status diagram
ClosedPublic

Authored by vlorentz on Fri, Oct 16, 11:01 AM.

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

vlorentz created this revision.Fri, Oct 16, 11:01 AM
vlorentz edited the summary of this revision. (Show Details)Fri, Oct 16, 11:01 AM

Build is green

Patch application report for D4283 (id=15141)

Could not rebase; Attempt merge onto 7e05dbe497...

Updating 7e05dbe4..7cc8d329
Fast-forward
 docs/images/status.png        | Bin 9825 -> 0 bytes
 docs/images/status.uml        |  27 +++++++++++++++++++++++++++
 docs/specs/spec-technical.rst |   6 +-----
 3 files changed, 28 insertions(+), 5 deletions(-)
 delete mode 100644 docs/images/status.png
 create mode 100644 docs/images/status.uml
Changes applied before test
commit 7cc8d3299e37c0c9fc916b31ab96931b48f7a6bc
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Fri Oct 16 11:00:49 2020 +0200

    spec-technical: Add 'partial' self-loop + annotate transitions in the status diagram

commit efda75628fb1ac9330779259ecd244717596c0a9
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Fri Oct 16 10:38:49 2020 +0200

    spec-technical: Rewrite state diagram using Plantuml.

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

ardumont requested changes to this revision.Fri, Oct 16, 1:17 PM
ardumont added a subscriber: ardumont.

Nice.

docs/images/status.uml
8

It's:

without In-Progress or 
with In-Progress: False
11

Unsure we want to enter into that much details.
We can modify the deposit with more than this IRI (iirc).

This revision now requires changes to proceed.Fri, Oct 16, 1:17 PM
vlorentz added inline comments.Fri, Oct 16, 1:19 PM
docs/images/status.uml
8

isn't it equivalent to what I wrote (except longer)?

11

then we'll add other transitions

ardumont added inline comments.Fri, Oct 16, 1:38 PM
docs/images/status.uml
11

It's not per say other transitions, it's just the same arrow with different actions which takes place underneath.
The transition stays the same nonetheless.

ardumont accepted this revision.Fri, Oct 16, 1:43 PM

Thanks.

You can add the other endpoints in another diff if you want ;)

Cheers,

This revision is now accepted and ready to land.Fri, Oct 16, 1:43 PM