Skip to content

add time_daemon component requirements - #153

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

add time_daemon component requirements#153
ryan-steel wants to merge 8 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: a6d29a6f-241b-4e4e-9bb8-42894fb4736b
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 (34 packages loaded, 10 targets configured)

Analyzing: target //:license-check (95 packages loaded, 142 targets configured)

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

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

Analyzing: target //:license-check (164 packages loaded, 8752 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 (168 packages loaded, 11172 targets configured)

INFO: Analyzed target //:license-check (170 packages loaded, 11422 targets configured).
[1 / 13] [Prepa] Creating source manifest for //:license.check.license_check ... (2 actions, 0 running)
[12 / 16] JavaToolchainCompileClasses external/rules_java+/toolchains/platformclasspath_classes; 0s disk-cache, processwrapper-sandbox ... (2 actions running)
INFO: From Generating Dash formatted dependency file ...:
INFO: Successfully converted 2 packages from Cargo.lock to bazel-out/k8-fastbuild/bin/formatted.txt
[14 / 16] JavaToolchainCompileBootClasspath external/rules_java+/toolchains/platformclasspath.jar; 0s disk-cache, processwrapper-sandbox
[16 / 16] no actions running
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.483s, Critical Path: 2.44s
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 Outdated
Comment thread score/time_daemon/docs/architecture/component_architecture.rst Outdated
Comment thread score/time_daemon/docs/requirements/chklst_req_inspection.rst
Comment thread score/time_daemon/docs/requirements/requirements.rst Outdated
@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 2 times, most recently from 2c79177 to 210b4ce Compare August 13, 2026 10:25
@ryan-steel
ryan-steel marked this pull request as draft August 13, 2026 11:57
Comment thread score/time_daemon/docs/requirements/requirements.rst Outdated
Comment thread score/time_daemon/docs/requirements/requirements.rst Outdated
Comment thread score/time_daemon/docs/architecture/component_architecture.rst Outdated
Comment thread score/time_daemon/docs/requirements/requirements.rst Outdated
@ryan-steel
ryan-steel force-pushed the add-time_daemon-component-requirements branch from efa1f9b to f379722 Compare August 14, 2026 09:54
@ryan-steel
ryan-steel force-pushed the add-time_daemon-component-requirements branch from f379722 to 937ff0c Compare August 14, 2026 10:05
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.

3 participants