Skip to content

docs: Add requirements for vajson#175

Merged
4og merged 12 commits into
eclipse-score:mainfrom
JuriSchroeder:vajson-docs-requirements
May 13, 2026
Merged

docs: Add requirements for vajson#175
4og merged 12 commits into
eclipse-score:mainfrom
JuriSchroeder:vajson-docs-requirements

Conversation

@JuriSchroeder
Copy link
Copy Markdown
Contributor

Taking over from here: eclipse-score/score#2358

@4og
Copy link
Copy Markdown
Member

4og commented May 6, 2026

@JuriSchroeder JuriSchroeder temporarily deployed to workflow-approval May 7, 2026 07:49 — with GitHub Actions Inactive
@JuriSchroeder JuriSchroeder temporarily deployed to workflow-approval May 7, 2026 07:49 — with GitHub Actions Inactive
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 7, 2026

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

@JuriSchroeder JuriSchroeder marked this pull request as ready for review May 7, 2026 09:28
@JuriSchroeder JuriSchroeder requested review from 4og and antonkri as code owners May 7, 2026 09:28
@JuriSchroeder JuriSchroeder temporarily deployed to workflow-approval May 7, 2026 12:30 — with GitHub Actions Inactive
@JuriSchroeder JuriSchroeder temporarily deployed to workflow-approval May 7, 2026 12:30 — with GitHub Actions Inactive
@4og 4og requested a review from aschemmel-tech May 7, 2026 13:23
Copy link
Copy Markdown

@aschemmel-tech aschemmel-tech left a comment

Choose a reason for hiding this comment

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

see inline comments

Comment thread docs/json/vajson/architecture/index.rst
Comment thread docs/json/vajson/requirements/index.rst
Comment thread docs/json/vajson/requirements/index.rst Outdated
Comment thread docs/json/vajson/requirements/index.rst Outdated
Comment thread docs/json/vajson/requirements/index.rst Outdated
Comment thread docs/json/vajson/requirements/index.rst Outdated
Comment thread docs/json/vajson/index.rst Outdated
Comment thread docs/json/vajson/index.rst Outdated
Safety-relevant component.


**Safety Related Functionality Required From Other Components**
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

These should be formulated as AoUs (and added to the requirements document). Is this the complete list of AoU?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I referenced the AoUs from the JSON module here. The list is not complete and has to be derived from the Vector internal safety artifacts, which I will dump in the future.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

ok

Comment thread docs/json/vajson/index.rst Outdated
Copy link
Copy Markdown

@aschemmel-tech aschemmel-tech left a comment

Choose a reason for hiding this comment

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

Findings resolved

@github-project-automation github-project-automation Bot moved this from In Progress to On Hold in BAS - Baselibs FT May 13, 2026
@4og 4og added this pull request to the merge queue May 13, 2026
Merged via the queue into eclipse-score:main with commit 729321d May 13, 2026
17 checks passed
@github-project-automation github-project-automation Bot moved this from On Hold to Done in BAS - Baselibs FT May 13, 2026
@4og 4og added the comp-json Related to score/json component label May 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp-json Related to score/json component

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants