Skip to content

test: add central S-Core PR checks workflow#346

Open
AlexanderLanin wants to merge 2 commits intomainfrom
on_pr
Open

test: add central S-Core PR checks workflow#346
AlexanderLanin wants to merge 2 commits intomainfrom
on_pr

Conversation

@AlexanderLanin
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown
Member Author

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

Copy link
Copy Markdown

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

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: Backlog

Development

Successfully merging this pull request may close these issues.

2 participants