Skip to content

Conversation

@AlexanderLanin
Copy link
Member

This is just preparation. It does not yet replace any of the other workflows.

We can reduce the number of workflows once this central on_pr workflows handles some of them.

@github-actions
Copy link

github-actions bot commented Jan 15, 2026

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run //src:license-check

Status: ⚠️ Needs Review

Click to expand output
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.3.0) and connecting to it...
INFO: Invocation ID: 17dabc05-497f-4858-b8de-978329d549ce
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
    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 (70 packages loaded, 9 targets configured)

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

Analyzing: target //src:license-check (123 packages loaded, 2293 targets configured)

Analyzing: target //src:license-check (132 packages loaded, 2458 targets configured)

Analyzing: target //src:license-check (134 packages loaded, 2484 targets configured)

INFO: Analyzed target //src:license-check (137 packages loaded, 4500 targets configured).
[9 / 13] JavaToolchainCompileClasses external/rules_java+/toolchains/platformclasspath_classes; 0s disk-cache
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: 14.634s, Critical Path: 0.31s
INFO: 13 processes: 4 disk cache hit, 9 internal.
INFO: Build completed successfully, 13 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>]

@AlexanderLanin
Copy link
Member Author

oh great... guess I need to fix that central workflow first 🤣

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a new centralized GitHub Actions workflow for S-Core PR checks as a preparatory step toward consolidating multiple existing workflows. The workflow calls a reusable workflow from the eclipse-score/cicd-workflows repository.

Changes:

  • Add new score_on_pr.yml workflow that serves as a central entry point for PR checks
  • Configure the workflow to trigger on pull requests, merge groups, and manual dispatch
  • Set up appropriate permissions for automated PR comment creation

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +36 to +37
# Note: "Common" is what will appear on the website as the job name.
Common:
Copy link

Copilot AI Jan 15, 2026

Choose a reason for hiding this comment

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

The job name 'PR-Checks' uses inconsistent naming convention. Other jobs in the repository use kebab-case (e.g., 'copyright-check', 'formatting-check', 'docs-verify'). Consider renaming to 'pr-checks' to maintain consistency with the existing naming patterns.

Suggested change
# Note: "Common" is what will appear on the website as the job name.
Common:
# Note: the job identifier uses kebab-case to match repository convention.
pr-checks:

Copilot uses AI. Check for mistakes.
@github-actions
Copy link

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

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

Labels

None yet

Projects

Status: Draft

Development

Successfully merging this pull request may close these issues.

2 participants