Skip to content

Fix test yaml repo cloning for test_and_docs workflow#470

Merged
MaximilianSoerenPollak merged 2 commits intoeclipse-score:mainfrom
MaximilianSoerenPollak:MSP_fix_testyaml
Apr 8, 2026
Merged

Fix test yaml repo cloning for test_and_docs workflow#470
MaximilianSoerenPollak merged 2 commits intoeclipse-score:mainfrom
MaximilianSoerenPollak:MSP_fix_testyaml

Conversation

@MaximilianSoerenPollak
Copy link
Copy Markdown
Contributor

@MaximilianSoerenPollak MaximilianSoerenPollak commented Apr 7, 2026

📌 Description

In the current workflow test_and_docs the workflow is on 'pull_request_target' which means that it takes the workflow of the main branch and runs that.
However, it also can mean if the checkout is done wrongly that the content is taken from main too.
The workflow that is called dictates the 'enviroment' in this case test_and_docs was running on pull_request_target and even though the 'test' workflow that it calls in a seperate step is running on 'pull_request' this is ignored and 'pull_request_target' behaviour is forced upon it.
With this in mind the test workflow did not work correctly and took the CONTENT from the main branch as the checkout was missconfigured.

This PR solves this issue.

🚨 Impact Analysis

  • This change does not violate any tool requirements and is covered by existing tool requirements
  • This change does not violate any design decisions
  • Otherwise I have created a ticket for new tool qualification

✅ Checklist

  • Added/updated documentation for new or changed features
  • Added/updated tests to cover the changes
  • Followed project coding standards and guidelines

@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 12:11:26 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: d6de59c5-cb9e-4a2f-bbdf-3a9f0e77f1cf
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
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
Loading: 0 packages loaded
    currently loading: src
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 (29 packages loaded, 9 targets configured)

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

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

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

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

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

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

Analyzing: target //src:license-check (147 packages loaded, 7587 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 (153 packages loaded, 7747 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).
[11 / 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 ... (2 actions running)
[14 / 16] JavaToolchainCompileBootClasspath external/rules_java+/toolchains/platformclasspath.jar; 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: 31.001s, Critical Path: 2.69s
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

@MaximilianSoerenPollak MaximilianSoerenPollak merged commit 3815e8b 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.

3 participants