Skip to content

ci: fix docs build in PRs - #558

Closed
AlexanderLanin wants to merge 1 commit into
eclipse-score:mainfrom
etas-contrib:pr-docs
Closed

ci: fix docs build in PRs#558
AlexanderLanin wants to merge 1 commit into
eclipse-score:mainfrom
etas-contrib:pr-docs

Conversation

@AlexanderLanin

@AlexanderLanin AlexanderLanin commented Aug 28, 2026

Copy link
Copy Markdown
Member

Unfortunately I need to abandon this half way through for today

Resolves #557

@github-actions

Copy link
Copy Markdown
Contributor

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.7.0) and connecting to it...
INFO: Invocation ID: 79d5745d-7aca-4f9a-bc71-4024ba31dc1f
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: 
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)

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

Analyzing: target //:license-check (136 packages loaded, 688 targets configured)

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

Analyzing: target //:license-check (162 packages loaded, 9365 targets configured)

Analyzing: target //:license-check (168 packages loaded, 9417 targets configured)

Analyzing: target //:license-check (169 packages loaded, 9541 targets configured)

INFO: Analyzed target //:license-check (173 packages loaded, 11555 targets configured).
[9 / 16] JavaToolchainCompileClasses external/rules_java+/toolchains/platformclasspath_classes; 0s disk-cache ... (2 actions, 1 running)
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: 21.818s, Critical Path: 0.52s
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>]

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.

We are waiting for merge queue to be enabled by eclipse-score/.eclipsefdn#214, after that the summary job in on-pr.yml will be the only one which is marked as required.

Therefore I would recommend adding these jobs to on-pr.yml and making the summary job depend on them, otherwise the merge queue will ignore the docs build.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

problems:

  • extend on-pr.yml to be triggered on release
  • documentation publish would depend on the slowest job in on-pr, instead of publishing as soon as documentation is available

alterative:

  • add docs build to .eclipsefdn as well

@MaciejKaszynski

Copy link
Copy Markdown
Contributor

Fixed on #561

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Doc Build should happen on-pr

3 participants