Skip to content

known_good.json maybe use "version" instead of "hash"#471

Merged
MaximilianSoerenPollak merged 2 commits intoeclipse-score:mainfrom
etas-contrib:hash-or-version
Apr 8, 2026
Merged

known_good.json maybe use "version" instead of "hash"#471
MaximilianSoerenPollak merged 2 commits intoeclipse-score:mainfrom
etas-contrib:hash-or-version

Conversation

@a-zw
Copy link
Copy Markdown
Contributor

@a-zw a-zw commented Apr 7, 2026

📌 Description

The known_good.json file may use "version" instead of "hash" and that must be supported.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 7, 2026

License Check Results

🚀 The license check job ran with the Bazel command:

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

Status: ⚠️ Needs Review

Click to expand output
[License Check Output]
2026/04/07 15:27:32 Downloading https://releases.bazel.build/8.4.2/release/bazel-8.4.2-linux-x86_64...
Extracting Bazel installation...
Starting local Bazel server (8.4.2) and connecting to it...
INFO: Invocation ID: 5a4da618-326c-4287-a537-ce005d814e99
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: src
Loading: 0 packages loaded
    currently loading: src
Loading: 0 packages loaded
    currently loading: src
Loading: 0 packages loaded
    currently loading: src
Analyzing: target //src:license-check (1 packages loaded)
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 (56 packages loaded, 9 targets configured)

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

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

Analyzing: target //src:license-check (127 packages loaded, 648 targets configured)

Analyzing: target //src:license-check (140 packages loaded, 3462 targets configured)

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

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

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

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

Analyzing: target //src:license-check (155 packages loaded, 9512 targets configured)

Analyzing: target //src:license-check (155 packages loaded, 9512 targets configured)

Analyzing: target //src:license-check (155 packages loaded, 9512 targets configured)

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

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

INFO: Analyzed target //src:license-check (157 packages loaded, 9762 targets configured).
[12 / 16] [Prepa] JavaToolchainCompileClasses external/rules_java+/toolchains/platformclasspath_classes
[13 / 16] JavaToolchainCompileClasses external/rules_java+/toolchains/platformclasspath_classes; 0s disk-cache, processwrapper-sandbox
[15 / 16] 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: 32.002s, Critical Path: 2.56s
INFO: 16 processes: 12 internal, 3 processwrapper-sandbox, 1 worker.
INFO: Build completed successfully, 16 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
Copy Markdown

github-actions bot commented Apr 7, 2026

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

Copy link
Copy Markdown
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.

Looks good.

Seems like we only needed a small changed indeed.:

only one comment though that has not much to do with the PR itself.

@MaximilianSoerenPollak MaximilianSoerenPollak merged commit 4b182d2 into eclipse-score:main Apr 8, 2026
14 checks passed
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