-
Notifications
You must be signed in to change notification settings - Fork 25
rename relations for SysML compatibility #446
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
2555455
815a36b
a5ff38b
d934910
954968a
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -292,9 +292,15 @@ needs_types: | |
| status: ^(valid|invalid)$ | ||
| # req-Id: tool_req__docs_common_attr_description | ||
| content: ^[\s\S]+$ | ||
| mandatory_links: | ||
| optional_links: | ||
|
RolandJentschETAS marked this conversation as resolved.
|
||
| # req-Id: tool_req__docs_req_link_satisfies_allowed | ||
| satisfies: stkh_req | ||
| # TODO: make derived_from mandatory once downstream repos migrate from satisfies | ||
| derived_from: stkh_req | ||
| satisfies: stkh_req # for backward compatibility | ||
| belongs_to: feat # for evaluation | ||
| satisfied_by: feat # new preferred name for belongs_to | ||
| # req-Id: tool_req__docs_req_link_covers_aou | ||
| covers: aou_req | ||
| optional_options: | ||
| codelink: ^.*$ | ||
| testlink: ^.*$ | ||
|
|
@@ -306,10 +312,6 @@ needs_types: | |
| # req-Id: tool_req__docs_req_attr_validity_correctness | ||
| valid_from: ^v(0|[1-9]\d*)\.(0|[1-9]\d*)(\.(0|[1-9]\d*))?$ | ||
| valid_until: ^v(0|[1-9]\d*)\.(0|[1-9]\d*)(\.(0|[1-9]\d*))?$ | ||
| optional_links: | ||
| belongs_to: feat # for evaluation | ||
| # req-Id: tool_req__docs_req_link_covers_aou | ||
| covers: aou_req | ||
|
MaximilianSoerenPollak marked this conversation as resolved.
|
||
| tags: | ||
| - requirement | ||
| - requirement_excl_process | ||
|
|
@@ -329,10 +331,16 @@ needs_types: | |
| status: ^(valid|invalid)$ | ||
| # req-Id: tool_req__docs_common_attr_description | ||
| content: ^[\s\S]+$ | ||
| mandatory_links: | ||
| optional_links: | ||
|
MaximilianSoerenPollak marked this conversation as resolved.
|
||
| # req-Id: tool_req__docs_req_link_satisfies_allowed | ||
| satisfies: feat_req | ||
| belongs_to: comp | ||
| # TODO: make derived_from mandatory once downstream repos migrate from satisfies | ||
| derived_from: feat_req | ||
| satisfies: feat_req # for backward compatibility | ||
| belongs_to: comp # for backward compatibility | ||
| satisfied_by: comp # new preferred name for belongs_to | ||
| # TODO: make satisfied_by mandatory once downstream repos migrate from belongs_to | ||
| # req-Id: tool_req__docs_req_link_covers_aou | ||
| covers: aou_req | ||
| optional_options: | ||
| codelink: ^.*$ | ||
| testlink: ^.*$ | ||
|
|
@@ -341,9 +349,6 @@ needs_types: | |
| # req-Id: tool_req__docs_req_attr_testcov | ||
| testcovered: ^(YES|NO)$ | ||
| hash: ^.*$ | ||
| optional_links: | ||
| # req-Id: tool_req__docs_req_link_covers_aou | ||
| covers: aou_req | ||
| tags: | ||
| - requirement | ||
| - requirement_excl_process | ||
|
|
@@ -986,9 +991,9 @@ needs_extra_links: | |
| # S-CORE Metamodel | ||
| ############################################################## | ||
| # Requirements | ||
| satisfies: | ||
| incoming: satisfied by | ||
| outgoing: satisfies | ||
| derived_from: | ||
| incoming: derives | ||
| outgoing: derived from | ||
|
|
||
| # req-Id: tool_req__docs_req_link_covers_aou | ||
| covers: | ||
|
|
@@ -1004,6 +1009,15 @@ needs_extra_links: | |
| incoming: has | ||
| outgoing: belongs to | ||
|
|
||
| satisfied_by: | ||
| incoming: satisfies | ||
| outgoing: satisfied by | ||
|
|
||
| satisfies: | ||
| incoming: satisfied by | ||
| outgoing: satisfies | ||
|
Comment on lines
+1016
to
+1018
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Should be deprecated in a new PR after this one
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Agreed β tracked in follow-up issue #594 |
||
|
|
||
| # kept for backward compatibility - old name for satisfies | ||
| fulfils: | ||
| incoming: fulfilled by | ||
| outgoing: fulfils | ||
|
|
@@ -1067,8 +1081,9 @@ graph_checks: | |
| include: stkh_req, feat_req, comp_req, aou_req, gd_req, tool_req | ||
| condition: safety == QM | ||
| check: | ||
| derived_from: safety == QM | ||
| satisfies: safety == QM | ||
| explanation: QM requirements cannot satisfy ASIL requirements. | ||
| explanation: QM requirements cannot be derived from ASIL requirements. | ||
|
MaximilianSoerenPollak marked this conversation as resolved.
|
||
|
|
||
| # req-Id: tool_req__docs_arch_link_qm_to_safety_req | ||
| tool_req__docs_arch_link_qm_to_safety_req: | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -129,7 +129,7 @@ def _strip_score_docs_overrides(content: str) -> str: | |
| # ^bazel_dep\(name = ["']score_docs_as_code["'] β literal call with either quote style | ||
| # (?:, version = ["'][^"']+["'])? β optional version argument, any value | ||
| # \) β closing paren | ||
| _BAZEL_DEP_PATTERN = r"""^bazel_dep\(name = ["']score_docs_as_code["'](?:, version = ["'][^"']+["'])?\)""" | ||
| _BAZEL_DEP_PATTERN = r"""^bazel_dep\(name = ["']score_docs_as_code["'][^)]*\)""" | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why is this suddenly changed? |
||
|
|
||
|
|
||
| def _write_module_bazel( | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please do not update bazel dependencies if not absolutely needed for your PR.