Skip to content

test: fully verify common status/security attribute requirements - #906

Merged
a-zw merged 4 commits into
eclipse-score:mainfrom
etas-contrib:fully-verify-common-attr-status-security
Oct 7, 2026
Merged

a-zw merged 4 commits into
eclipse-score:mainfrom
etas-contrib:fully-verify-common-attr-status-security

Conversation

@a-zw

@a-zw a-zw commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

📌 Description

Improves coverage.

🚨 Impact Analysis

  • This change does not violate any tool requirements and is covered by existing tool requirements
  • This change does not violate any design decisions
  • Otherwise I have created a ticket for new tool qualification

✅ Checklist

  • Added/updated documentation for new or changed features
  • Added/updated tests to cover the changes
  • Followed project coding standards and guidelines

@github-actions

github-actions Bot commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

Documentation preview for this pull request is available at:
pr-906: https://eclipse-score.github.io/docs-as-code/pr-906/

Comment thread src/extensions/score_metamodel/tests/rst/options/test_options_common_attrs.rst Outdated
Comment thread src/extensions/score_metamodel/tests/rst/options/test_options_common_attrs.rst Outdated
Comment thread src/extensions/score_metamodel/tests/rst/options/test_options_common_attrs.rst Outdated
@a-zw
a-zw force-pushed the fully-verify-common-attr-status-security branch from 6bd89cc to 12bc1ea Compare October 6, 2026 07:49

@MaximilianSoerenPollak MaximilianSoerenPollak left a comment

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.

JJust a comment, but that shouldn't be a deal breaker though something we think about.

@a-zw
a-zw merged commit 102aad3 into eclipse-score:main Oct 7, 2026
32 checks passed
@a-zw
a-zw deleted the fully-verify-common-attr-status-security branch October 7, 2026 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants