Skip to content

bugfix: linking of plat_saf_dfa#374

Merged
AlexanderLanin merged 2 commits intoeclipse-score:mainfrom
etas-contrib:fix_linking_plat_saf_dfa
Feb 2, 2026
Merged

bugfix: linking of plat_saf_dfa#374
AlexanderLanin merged 2 commits intoeclipse-score:mainfrom
etas-contrib:fix_linking_plat_saf_dfa

Conversation

@RolandJentschETAS
Copy link
Contributor

📌 Description

bugifx: linking of plat_saf_dfa to stake holder requirement instead of feature requirement. See gd_req__saf_attr_requirements_check

🚨 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
Copy link

github-actions bot commented Feb 2, 2026

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run //src:license-check

Status: ⚠️ Needs Review

Click to expand output
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.3.0) and connecting to it...
INFO: Invocation ID: 96573216-cf95-4da7-a3b2-5f400fd33cfa
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: src
Loading: 0 packages loaded
    currently loading: src
Analyzing: target //src:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //src:license-check (1 packages loaded, 0 targets configured)

Analyzing: target //src:license-check (67 packages loaded, 9 targets configured)

Analyzing: target //src:license-check (72 packages loaded, 9 targets configured)

Analyzing: target //src:license-check (124 packages loaded, 825 targets configured)

Analyzing: target //src:license-check (126 packages loaded, 2370 targets configured)

Analyzing: target //src:license-check (138 packages loaded, 2444 targets configured)

Analyzing: target //src:license-check (138 packages loaded, 2456 targets configured)

Analyzing: target //src:license-check (138 packages loaded, 2456 targets configured)

Analyzing: target //src:license-check (141 packages loaded, 4466 targets configured)

Analyzing: target //src:license-check (143 packages loaded, 4716 targets configured)

INFO: Analyzed target //src:license-check (143 packages loaded, 4716 targets configured).
[9 / 13] JavaToolchainCompileClasses external/rules_java+/toolchains/platformclasspath_classes; 0s disk-cache, processwrapper-sandbox ... (2 actions running)
[11 / 13] JavaToolchainCompileBootClasspath external/rules_java+/toolchains/platformclasspath.jar; 0s disk-cache, processwrapper-sandbox
[12 / 13] Building src/license.check.license_check.jar (); 0s disk-cache, multiplex-worker
INFO: Found 1 target...
Target //src:license.check.license_check up-to-date:
  bazel-bin/src/license.check.license_check
  bazel-bin/src/license.check.license_check.jar
INFO: Elapsed time: 20.759s, Critical Path: 2.60s
INFO: 13 processes: 9 internal, 3 processwrapper-sandbox, 1 worker.
INFO: Build completed successfully, 13 total actions
INFO: Running command line: bazel-bin/src/license.check.license_check src/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
Copy link

github-actions bot commented Feb 2, 2026

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

@RolandJentschETAS RolandJentschETAS changed the title bugifx: linking of plat_saf_dfa bugfix: linking of plat_saf_dfa Feb 2, 2026
@AlexanderLanin
Copy link
Member

merge after consumer tests pass

@AlexanderLanin AlexanderLanin merged commit 6b35e02 into eclipse-score:main Feb 2, 2026
14 checks passed
@AlexanderLanin AlexanderLanin deleted the fix_linking_plat_saf_dfa branch February 2, 2026 09:01
Copy link
Contributor

@MaximilianSoerenPollak MaximilianSoerenPollak left a comment

Choose a reason for hiding this comment

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

Just a question

mandatory_links:
includes: ^logic_arc_int(_op)*__.+$
fulfils: feat_req
fulfils: feat_req # TODO: make it mandatory
Copy link
Contributor

Choose a reason for hiding this comment

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

This already is a mandatory link, is the TODO then not needed anymore?

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.

3 participants