Replace arch and req to S-CORE#2974
Conversation
PandaeDo
requested review from
a team,
FScholPer,
antonkri,
johannes-esr and
qor-lb
as code owners
|
The created documentation from the pull request is available at: docu-html |
| :status: valid | ||
| :belongs_to: feat__persistency | ||
|
|
||
| .. uml:: _assets/kvs_static_view.puml |
There was a problem hiding this comment.
Is there any reason to use puml instead of the integrated draw function of sphinx ? Outside the appearance (all other features look different) it does not allow the automated drawing, but needs manual created drawings and needs manual checks.
There was a problem hiding this comment.
I don't know the exact reason. But I guess this was the stand of knowledge the authors had as the initially created the architecture over one year ago. But I take this as a point for the feature team meeting next week. We have some changes and will plan to update documents. This might be a good chance also to update the architecture as you described.
There was a problem hiding this comment.
now not part of the change any more
aschemmel-tech
left a comment
aschemmel-tech
left a comment
There was a problem hiding this comment.
not all architecture elements are needed on score repo, see inline comments
aschemmel-tech
left a comment
aschemmel-tech
left a comment
There was a problem hiding this comment.
old findings fixed, but found new ones
PandaeDo
force-pushed
the
vohae_persistency_documents
branch
from
fb19703 to
7917638
Compare
aschemmel-tech
left a comment
aschemmel-tech
left a comment
There was a problem hiding this comment.
findings resolved now
5 participants
After clarification in process community today, documents of architecture and requirements, also the feature request will be placed again in S-CORE. #2964