Skip to content

Use valid_inspected state in reqs - #3341

Open
aschemmel-tech wants to merge 1 commit into
mainfrom
aschemmel-tech-valid-inspected-state
Open

aschemmel-tech wants to merge 1 commit into
mainfrom
aschemmel-tech-valid-inspected-state

Conversation

@aschemmel-tech

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

Copy link
Copy Markdown
Contributor

Documentation preview for this pull request is available at:
pr-3341: https://eclipse-score.github.io/score/pr-3341/

Comment thread docs/features/persistency/requirements/chklst_req_inspection.rst
@aschemmel-tech
aschemmel-tech marked this pull request as draft October 15, 2026 13:45
@aschemmel-tech
aschemmel-tech force-pushed the aschemmel-tech-valid-inspected-state branch from 2d344e2 to 7cf255c Compare October 15, 2026 13:54
@aschemmel-tech
aschemmel-tech marked this pull request as ready for review October 15, 2026 13:55
:valid_from: v2.0.0

The logging :term:`framework` shall be classified according to the overall system's safety concept if logging information is part of the verification strategy.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It is not recommended to use extent functionality. Better to add this to any element. Additionally some elements already contains such tags.

@aschemmel-tech aschemmel-tech Oct 16, 2026 •

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 understand that we do not want to use this across files (and I think this is even a banned function). But within one file I do not see a problem, but the benefit of lower effort and no risk to forget a label. No "logging" tag is duplicated (see the generated picture in the checklist). Also this is the solution which is currently in the module_template

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog
Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants