Skip to content

add time_daemon component requirements - #153

Open
ryan-steel wants to merge 4 commits into
eclipse-score:mainfrom
etas-contrib:add-time_daemon-component-requirements
Open

add time_daemon component requirements#153
ryan-steel wants to merge 4 commits into
eclipse-score:mainfrom
etas-contrib:add-time_daemon-component-requirements

Conversation

@ryan-steel

Copy link
Copy Markdown
Contributor

Improvement

Description

Add initial component documentation for time_daemon following S-CORE process:

  • Component requirements (docs/requirements/)
  • Architecture boilerplate (docs/architecture/) to give us valid needs to link against (comp__time_daemon) - these are the template files from module_template, the only changes I've made were to update IDs, the rest of the content should be filled in a future PR.

All status: invalid (draft) pending review.

Important

The added docs won't be built until the changes in #148 for updating to docs_as_code 6.0 and using docs bundles is merged. I've run the build locally with those changes implemented and the docs build without error.

Related ticket

Part of #95

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run //:license-check

Status: ⚠️ Needs Review

Click to expand output
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.6.0) and connecting to it...
INFO: Invocation ID: fbff4b0e-5d75-4f97-95ab-f04ee3914755
Computing main repo mapping: 
WARNING: /home/runner/work/time/time/MODULE.bazel:13:7: The attribute 'compatibility_level' in module() is a no-op and will be removed in a future Bazel release. Please remove it from your MODULE.bazel file.
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 3 packages loaded
Loading: 3 packages loaded
    currently loading: 
Loading: 3 packages loaded
    currently loading: 
Loading: 3 packages loaded
    currently loading: 
Loading: 3 packages loaded
    currently loading: 
Loading: 3 packages loaded
    currently loading: 
Analyzing: target //:license-check (4 packages loaded, 0 targets configured)
Analyzing: target //:license-check (4 packages loaded, 0 targets configured)

Analyzing: target //:license-check (48 packages loaded, 10 targets configured)

Analyzing: target //:license-check (87 packages loaded, 10 targets configured)

Analyzing: target //:license-check (144 packages loaded, 3098 targets configured)

Analyzing: target //:license-check (152 packages loaded, 6476 targets configured)

Analyzing: target //:license-check (157 packages loaded, 9225 targets configured)

Analyzing: target //:license-check (164 packages loaded, 9276 targets configured)

Analyzing: target //:license-check (165 packages loaded, 9284 targets configured)

Analyzing: target //:license-check (165 packages loaded, 9284 targets configured)

Analyzing: target //:license-check (165 packages loaded, 9284 targets configured)

Analyzing: target //:license-check (169 packages loaded, 11296 targets configured)

INFO: Analyzed target //:license-check (170 packages loaded, 11422 targets configured).
[11 / 16] Creating runfiles tree bazel-out/k8-opt-exec-ST-d57f47055a04/bin/external/score_tooling+/dash/tool/formatters/dash_format_converter.runfiles [for tool]; 0s local
INFO: From Generating Dash formatted dependency file ...:
INFO: Successfully converted 2 packages from Cargo.lock to bazel-out/k8-fastbuild/bin/formatted.txt
[13 / 16] JavaToolchainCompileClasses external/rules_java+/toolchains/platformclasspath_classes; 0s disk-cache, processwrapper-sandbox
[14 / 16] JavaToolchainCompileBootClasspath external/rules_java+/toolchains/platformclasspath.jar; 0s disk-cache, processwrapper-sandbox
INFO: Found 1 target...
Target //:license.check.license_check up-to-date:
  bazel-bin/license.check.license_check
  bazel-bin/license.check.license_check.jar
INFO: Elapsed time: 29.273s, Critical Path: 2.65s
INFO: 16 processes: 12 internal, 3 processwrapper-sandbox, 1 worker.
INFO: Build completed successfully, 16 total actions
INFO: Running command line: bazel-bin/license.check.license_check ./formatted.txt <args omitted>
usage: org.eclipse.dash.licenses.cli.Main [-batch <int>] [-cd <url>]
       [-confidence <int>] [-ef <url>] [-excludeSources <sources>] [-help] [-lic
       <url>] [-project <shortname>] [-repo <url>] [-review] [-summary <file>]
       [-timeout <seconds>] [-token <token>]

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

The created documentation from the pull request is available at: docu-html

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds initial S-CORE process documentation scaffolding for the score/time_daemon component, establishing requirement artifacts and architecture boilerplate to support traceability and future inspection.

Changes:

  • Added component requirements document and requirements inspection checklist under score/time_daemon/docs/requirements/.
  • Added architecture boilerplate and architecture inspection checklist under score/time_daemon/docs/architecture/.
  • Added component-level docs entry point and toctree wiring for requirements + architecture.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 8 comments.

Show a summary per file
File Description
score/time_daemon/docs/requirements/requirements.rst Introduces initial comp_req entries for time_daemon (draft/invalid baseline).
score/time_daemon/docs/requirements/index.rst Adds requirements section toctree for the component docs.
score/time_daemon/docs/requirements/chklst_req_inspection.rst Adds requirements inspection checklist template for the component.
score/time_daemon/docs/index.rst Adds the component documentation landing page linking requirements and architecture.
score/time_daemon/docs/architecture/index.rst Adds architecture section toctree for the component docs.
score/time_daemon/docs/architecture/component_architecture.rst Adds component architecture boilerplate, including comp and static/dynamic view placeholders.
score/time_daemon/docs/architecture/chklst_arc_inspection.rst Adds architecture inspection checklist template for the component.
Suppressed comments (2)

score/time_daemon/docs/architecture/component_architecture.rst:102

  • The "comp_arc_dyn" need has an empty ":fulfils:" option. If requirement allocation is not decided yet, avoid empty relation fields to prevent Sphinx-Needs warnings/errors.
   :belongs_to: comp__time_daemon
   :fulfils:

score/time_daemon/docs/architecture/chklst_arc_inspection.rst:178

  • Extra quote characters in the note: ""n/a"" should be "n/a".
Note: If a Review ID is not applicable for your architecture, then state ""n/a" in status and comment accordingly in remarks.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread score/time_daemon/docs/index.rst Outdated
Comment thread score/time_daemon/docs/architecture/component_architecture.rst Outdated
Comment thread score/time_daemon/docs/architecture/component_architecture.rst Outdated
Comment thread score/time_daemon/docs/architecture/component_architecture.rst Outdated
Comment thread score/time_daemon/docs/architecture/chklst_arc_inspection.rst
Comment thread score/time_daemon/docs/architecture/component_architecture.rst
Comment thread score/time_daemon/docs/requirements/chklst_req_inspection.rst
Comment thread score/time_daemon/docs/requirements/requirements.rst
@ryan-steel
ryan-steel force-pushed the add-time_daemon-component-requirements branch from 20e33f8 to 2c79177 Compare August 6, 2026 15:11
@ryan-steel
ryan-steel force-pushed the add-time_daemon-component-requirements branch from 55625e2 to 2c79177 Compare August 7, 2026 13:53
@ryan-steel
ryan-steel force-pushed the add-time_daemon-component-requirements branch from 333e497 to 2c79177 Compare August 7, 2026 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

2 participants