Skip to content

Use score::mw::log#191

Open
MaciejKaszynski wants to merge 11 commits into
eclipse-score:mainfrom
etas-contrib:log-internal-types
Open

Use score::mw::log#191
MaciejKaszynski wants to merge 11 commits into
eclipse-score:mainfrom
etas-contrib:log-internal-types

Conversation

@MaciejKaszynski
Copy link
Copy Markdown
Contributor

  • Use score logging in tests
  • Left use_cout_logger as a flag for debugging

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 5, 2026

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run --lockfile_mode=error //:license-check

Status: ⚠️ Needs Review

Click to expand output
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.4.2) and connecting to it...
INFO: Invocation ID: 1cf09246-c459-4df3-a228-c5685a1b97bc
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)

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

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

Analyzing: target //:license-check (137 packages loaded, 2708 targets configured)

Analyzing: target //:license-check (148 packages loaded, 5470 targets configured)

Analyzing: target //:license-check (152 packages loaded, 8045 targets configured)

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

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

INFO: Analyzed target //:license-check (162 packages loaded, 10232 targets configured).
[9 / 16] [Prepa] Expanding template external/score_tooling+/dash/tool/formatters/dash_format_converter [for tool]
[14 / 16] [Prepa] JavaToolchainCompileBootClasspath external/rules_java+/toolchains/platformclasspath.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: 20.479s, Critical Path: 2.40s
INFO: 16 processes: 12 internal, 3 processwrapper-sandbox, 1 worker.
INFO: Build completed successfully, 16 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>]

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 5, 2026

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

Comment thread BUILD Outdated
Comment thread src/launch_manager_daemon/src/process_group_manager/processlauncher.cpp Outdated
Comment thread src/launch_manager_daemon/src/process_group_manager/processlauncher.cpp Outdated
Comment thread src/launch_manager_daemon/src/process_group_manager/processlauncher.cpp Outdated
Comment thread src/launch_manager_daemon/common/include/score/lcm/identifier_hash.hpp Outdated
Comment thread src/launch_manager_daemon/common/include/score/lcm/internal/log.hpp Outdated
Comment thread src/launch_manager_daemon/common/include/score/lcm/internal/log.hpp Outdated
Comment thread src/launch_manager_daemon/common/include/score/lcm/identifier_hash.hpp Outdated
Comment thread tests/README.md
Comment thread tests/README.md Outdated
Comment thread tests/README.md
Comment thread src/launch_manager_daemon/src/process_group_manager/processlauncher.cpp Outdated
Comment thread src/launch_manager_daemon/src/process_group_manager/processlauncher.cpp Outdated
Copy link
Copy Markdown
Contributor

@paulquiring paulquiring left a comment

Choose a reason for hiding this comment

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

looks good now

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.

3 participants