Add ts client component requirements - #149
Conversation
License Check Results🚀 The license check job ran with the Bazel command: bazel run //:license-checkStatus: Click to expand output |
2956a19 to
1daac44
Compare
|
The created documentation from the pull request is available at: docu-html |
21fc11f to
2641242
Compare
There was a problem hiding this comment.
Pull request overview
This PR adds S-CORE process documentation for the score/ts_client component, primarily introducing component requirements plus the associated inspection/checklist scaffolding and initial architecture documentation structure.
Changes:
- Added component requirements and AoU requirements for the Time Sync Client (
ts_client) in Sphinx-needs format. - Added requirements and architecture toctrees and inspection checklist documents for
ts_client. - Updated component doc navigation (including a change to
time_slave’s docs index).
Reviewed changes
Copilot reviewed 8 out of 9 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| score/ts_client/docs/requirements/requirements.rst | New component requirements and AoU requirement definitions for ts_client. |
| score/ts_client/docs/requirements/index.rst | Requirements section toctree for the ts_client docs. |
| score/ts_client/docs/requirements/chklst_req_inspection.rst | Requirements inspection checklist content and needtables (currently includes template placeholders). |
| score/ts_client/docs/index.rst | Top-level ts_client component doc entry and links to requirements/architecture sections. |
| score/ts_client/docs/architecture/index.rst | Architecture section toctree for the ts_client docs. |
| score/ts_client/docs/architecture/component_architecture.rst | Initial component architecture document and comp / static+dynamic view needs. |
| score/ts_client/docs/architecture/chklst_arc_inspection.rst | Architecture inspection checklist document. |
| score/ts_client/docs/.gitkeep | Placeholder for docs directory. |
| score/time_slave/docs/index.rst | Removes multiple toctree entries from the time_slave docs index. |
Suppressed comments (3)
score/ts_client/docs/requirements/chklst_req_inspection.rst:48
- Typo in the inspection roles list: duplicated "is" in "who is is".
- moderator: <committer explicitly named here, who is is the safety manager, security manager or quality manager initiating the inspection>
score/time_slave/docs/index.rst:22
- This PR removes the time_slave docs toctree entries for architecture/requirements/manuals/etc., but that change isn’t mentioned in the PR description. If the intent is to temporarily omit these sections (e.g., because the pages don’t exist yet), add an explanatory note here (or move this change to a separate PR).
.. toctree::
:maxdepth: 1
component_classification
score/ts_client/docs/requirements/chklst_req_inspection.rst:187
- The AoU inspection needtable still uses the template placeholder "component_name" in both the filter and tag selector, so it will never match this component’s needs.
.. needtable::
:filter: "component_name" in docname and "requirements" in docname and docname is not None and status == "valid"
:style: table
:types: aou_req
:tags: component_name
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
ff3cbf3 to
117b34b
Compare
Adds component requirements documentation for ts_client following S-CORE requirements engineering process and module_template.
Note: Component definition (comp__time_ts_client) and architecture documentation not yet complete, so satisfied_by fields will create docs build errors if these docs are included at the moment.
Note, this doesn't include the changes required to include these docs in the docs build. Those changes are handled in #148 already and would rather avoid conflicts.
Changes:
Requirements follow QM safety level and NO security classification to match feature requirements. All requirements use time_ts_client module_component naming pattern.
Related ticket
Addresses part of #97