Skip to content

Conversation

@PiotrKorkus
Copy link
Contributor

Component requirements traceability visualizations

Bugfix

Important

Use this template only for bugfixes that do not influence topics covered by contribution requests or improvements.

Caution

Make sure to submit your pull-request as Draft until you are ready to have it reviewed by the Committers.

Description

[A short description of the bug being fixed by the contribution.]

Related ticket

Important

Please replace [ISSUE-NUMBER] with the issue-number that tracks this bug fix. If there is no such
ticket yet, create one via this issue template.

closes [ISSUE-NUMBER] (bugfix ticket)

@github-actions
Copy link

github-actions bot commented Jan 27, 2026

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run //:license-check

Status: ⚠️ Needs Review

Click to expand output
[License Check Output]
2026/01/28 16:08:36 Downloading https://releases.bazel.build/8.3.0/release/bazel-8.3.0-linux-x86_64...
Extracting Bazel installation...
Starting local Bazel server (8.3.0) and connecting to it...
INFO: Invocation ID: 48ed3f3c-e0f1-43f8-8465-6afa742ee132
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
DEBUG: Rule 'rust_qnx8_toolchain+' indicated that a canonical reproducible form can be obtained by modifying arguments integrity = "sha256-oEubHgeZDdT0svMmBKJx7c3/2TdSI/vfwRUyDn+TPGA="
DEBUG: Repository rust_qnx8_toolchain+ instantiated at:
  <builtin>: in <toplevel>
Repository rule http_archive defined at:
  /home/runner/.bazel/external/bazel_tools/tools/build_defs/repo/http.bzl:394:31: in <toplevel>
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 3 packages loaded
Loading: 3 packages loaded
    currently loading: 
Loading: 3 packages loaded
    currently loading: 
Analyzing: target //:license-check (4 packages loaded, 0 targets configured)
Analyzing: target //:license-check (4 packages loaded, 0 targets configured)

Analyzing: target //:license-check (81 packages loaded, 10 targets configured)

Analyzing: target //:license-check (87 packages loaded, 10 targets configured)

Analyzing: target //:license-check (145 packages loaded, 943 targets configured)

Analyzing: target //:license-check (153 packages loaded, 1836 targets configured)

Analyzing: target //:license-check (153 packages loaded, 1836 targets configured)

Analyzing: target //:license-check (156 packages loaded, 3597 targets configured)

Analyzing: target //:license-check (156 packages loaded, 3843 targets configured)

Analyzing: target //:license-check (157 packages loaded, 7871 targets configured)

Analyzing: target //:license-check (158 packages loaded, 7995 targets configured)

Analyzing: target //:license-check (160 packages loaded, 8879 targets configured)

Analyzing: target //:license-check (161 packages loaded, 8887 targets configured)

Analyzing: target //:license-check (161 packages loaded, 8887 targets configured)

Analyzing: target //:license-check (161 packages loaded, 8887 targets configured)

Analyzing: target //:license-check (161 packages loaded, 8887 targets configured)

INFO: Analyzed target //:license-check (162 packages loaded, 9013 targets configured).
INFO: From Generating Dash formatted dependency file ...:
INFO: Successfully converted 62 packages from Cargo.lock to bazel-out/k8-fastbuild/bin/formatted.txt
[12 / 14] [Prepa] JavaToolchainCompileBootClasspath external/rules_java+/toolchains/platformclasspath.jar
[13 / 14] [Prepa] Building license.check.license_check.jar ()
INFO: Found 1 target...
Target //:license.check.license_check up-to-date:
  bazel-bin/license.check.license_check
  bazel-bin/license.check.license_check.jar
INFO: Elapsed time: 37.154s, Critical Path: 2.57s
INFO: 14 processes: 9 internal, 4 processwrapper-sandbox, 1 worker.
INFO: Build completed successfully, 14 total actions
INFO: Running command line: bazel-bin/license.check.license_check ./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>]

@PiotrKorkus PiotrKorkus force-pushed the piotrkorkus_test_stats branch from 1a5c224 to d3ec34f Compare January 27, 2026 12:36
@github-actions
Copy link

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

@PiotrKorkus PiotrKorkus force-pushed the piotrkorkus_test_stats branch 2 times, most recently from 145b67d to e92b654 Compare January 27, 2026 14:28
PandaeDo
PandaeDo previously approved these changes Jan 28, 2026
Copy link
Contributor

@PandaeDo PandaeDo left a comment

Choose a reason for hiding this comment

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

Really like it. I would rephrase one sentence


Failed Tests

*Hint: this table is empty by definition, as PRs with failing tests are not allowed to be merged in persistency repo.*
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
*Hint: this table is empty by definition, as PRs with failing tests are not allowed to be merged in persistency repo.*
*Hint: Before a PR can be merged all tests have to be successful.*

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hmm - we can have xfail for discovered bugs before they are fixed. Im not sure if that is a "successful" test.

Copy link
Contributor

Choose a reason for hiding this comment

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

I just want to avoid that somebody is thinking that you define the table as always empty. Because you wrote that "this table is empty by definition".

Component requirements traceability visualizations
new docs with reports scan
Refer to `*_v2` needs
- move to pyproject.toml pytest config
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants