Skip to content

Changing ready condition reqs - #467

Open
MaciejKaszynski wants to merge 3 commits into
eclipse-score:mainfrom
etas-contrib:ready-conditions-reqs
Open

Changing ready condition reqs#467
MaciejKaszynski wants to merge 3 commits into
eclipse-score:mainfrom
etas-contrib:ready-conditions-reqs

Conversation

@MaciejKaszynski

Copy link
Copy Markdown
Contributor

Altering requirements in accordance to discussion on https://github.com/orgs/eclipse-score/discussions/2386#discussioncomment-17494192

@github-actions

github-actions Bot commented Aug 13, 2026

Copy link
Copy Markdown

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.6.0) and connecting to it...
INFO: Invocation ID: 998ff5bc-8c29-41eb-82b5-b41bbaa44fcb
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: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 1 packages loaded
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)

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

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

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

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

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

Analyzing: target //:license-check (170 packages loaded, 9397 targets configured)

Analyzing: target //:license-check (170 packages loaded, 9397 targets configured)

INFO: Analyzed target //:license-check (175 packages loaded, 11535 targets configured).
[7 / 16] Creating runfiles tree bazel-out/k8-opt-exec-ST-d57f47055a04/bin/external/score_tooling+/dash/tool/formatters/dash_format_converter.runfiles [for tool]; 0s local
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: 23.887s, Critical Path: 0.36s
INFO: 16 processes: 4 disk cache hit, 12 internal.
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>]

.. comp_req:: Conditional launch polling interval
:id: comp_req__launch_man__polling_interval
.. comp_req:: Conditionally launch of processes
:id: comp_req__launch_man__config_ready_cond

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.

Looks like duplicate ID...
Maybe comp_req__launch_man__ready_cond

Comment thread score/launch_manager/docs/requirements/requirements.rst Outdated
@github-actions

Copy link
Copy Markdown

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

@danth danth left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Some suggestions for the requirement titles.

Comment thread score/launch_manager/docs/requirements/requirements.rst Outdated
Comment thread score/launch_manager/docs/requirements/requirements.rst Outdated
Comment thread score/launch_manager/docs/requirements/requirements.rst Outdated
Comment thread score/launch_manager/docs/requirements/requirements.rst Outdated
Comment thread score/launch_manager/docs/requirements/requirements.rst Outdated
Co-authored-by: Daniel Thwaites <danth@danth.me>
Signed-off-by: Maciej Kaszynski <161459353+MaciejKaszynski@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

3 participants