Skip to content

build(coverage): move coverage to tools/coverage, add scope check - #577

Draft
arkjedrz wants to merge 1 commit into
mainfrom
arkjedrz_tooling-06-coverage
Draft

arkjedrz wants to merge 1 commit into
mainfrom
arkjedrz_tooling-06-coverage

Conversation

@arkjedrz

Copy link
Copy Markdown
Contributor
  • Move quality/coverage to tools/coverage, as in baselibs. Rewrite the //quality/coverage labels, the .bazelrc import, the comments in MODULE.bazel and BUILD, and the justifications path in coverage.yml and release.yml.
  • Align the llvm_cov config with baselibs. Set the platform and the LLVM and Ferrocene toolchains explicitly. Keep the warning feature disables and --instrumentation_filter=^//score[/:].
  • Add baselibs' check_coverage_scope.sh and scope_exclusions.txt. The check fails when a production library under //score/... is neither in //tools/coverage:coverage_scope nor excluded.
  • Replace coverage.yml with coverage_report.yml. Use baselibs' triggers (push, merge_group, workflow_call) and steps, including the scope check, on the cicd common steps with SHA pins. Keep workflow_dispatch and the _coverage_report artifact name.
  • Drop the cargo-coverage job from check.yml. coverage_report.yml covers rust_kvs. cpp-bazel stays until check.yml is replaced.
  • Document the scope check and coverage_report.yml in tools/coverage/README.md.

@arkjedrz arkjedrz self-assigned this Oct 23, 2026
@arkjedrz
arkjedrz added this pull request to stack #526 October 23, 2026 12:27
@github-actions

github-actions Bot commented Oct 23, 2026 •

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]
Starting local Bazel server (8.7.0) and connecting to it...
INFO: Invocation ID: ece7ff98-ddef-4e8f-9c99-3451b5ab49a0
Computing main repo mapping: 
Computing main repo mapping: 
WARNING: For repository 'score_docs_as_code', the root module requires module version score_docs_as_code@8.5.1, but got score_docs_as_code@8.6.0 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 2 packages loaded
Loading: 2 packages loaded
    currently loading: 
Loading: 2 packages loaded
    currently loading: 
Loading: 2 packages loaded
    currently loading: 
Loading: 2 packages loaded
    currently loading: 
Loading: 2 packages loaded
    currently loading: 
Loading: 2 packages loaded
    currently loading: 
Analyzing: target //:license-check (3 packages loaded, 0 targets configured)
Analyzing: target //:license-check (3 packages loaded, 0 targets configured)

Analyzing: target //:license-check (30 packages loaded, 14 targets configured)

Analyzing: target //:license-check (97 packages loaded, 14 targets configured)

Analyzing: target //:license-check (100 packages loaded, 14 targets configured)

Analyzing: target //:license-check (100 packages loaded, 14 targets configured)

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

Analyzing: target //:license-check (197 packages loaded, 6701 targets configured)

Analyzing: target //:license-check (199 packages loaded, 7666 targets configured)

Analyzing: target //:license-check (201 packages loaded, 7718 targets configured)

Analyzing: target //:license-check (205 packages loaded, 8916 targets configured)

Analyzing: target //:license-check (207 packages loaded, 11661 targets configured)

Analyzing: target //:license-check (207 packages loaded, 11661 targets configured)

Analyzing: target //:license-check (211 packages loaded, 17650 targets configured)

Analyzing: target //:license-check (211 packages loaded, 17650 targets configured)

Analyzing: target //:license-check (212 packages loaded, 17774 targets configured)

Analyzing: target //:license-check (212 packages loaded, 17774 targets configured)

Analyzing: target //:license-check (212 packages loaded, 17774 targets configured)

Analyzing: target //:license-check (212 packages loaded, 17774 targets configured)

Analyzing: target //:license-check (212 packages loaded, 17774 targets configured)

Analyzing: target //:license-check (212 packages loaded, 17774 targets configured)

Analyzing: target //:license-check (212 packages loaded, 17774 targets configured)

INFO: Analyzed target //:license-check (213 packages loaded, 17903 targets configured).
[12 / 16] JavaToolchainCompileClasses external/rules_java+/toolchains/platformclasspath_classes; 0s disk-cache, processwrapper-sandbox ... (2 actions, 1 running)
INFO: From Generating Dash formatted dependency file ...:
INFO: Successfully converted 50 packages from Cargo.lock to bazel-out/k8-fastbuild/bin/tools/formatted.txt
[14 / 16] JavaToolchainCompileBootClasspath external/rules_java+/toolchains/platformclasspath.jar; 0s disk-cache, processwrapper-sandbox
INFO: Found 1 target...
Target //tools:license.check.license_check up-to-date:
  bazel-bin/tools/license.check.license_check
  bazel-bin/tools/license.check.license_check.jar
INFO: Elapsed time: 51.401s, Critical Path: 2.97s
INFO: 16 processes: 12 internal, 3 processwrapper-sandbox, 1 worker.
INFO: Build completed successfully, 16 total actions
INFO: Running command line: bazel-bin/tools/license.check.license_check tools/formatted.txt <args omitted>
[main] INFO Querying Eclipse Foundation for license data for 42 items.
[main] INFO Found 32 items.
[main] INFO Querying ClearlyDefined for license data for 10 items.
[main] INFO Found 10 items.
[main] INFO License information could not be automatically verified for the following content:
[main] INFO 
[main] INFO crate/cratesio/-/lazy_static/1.5.1
[main] INFO 
[main] INFO This content is either not correctly mapped by the system, or requires review.

@github-actions

Copy link
Copy Markdown
Contributor

Documentation preview for this pull request is available at:
pr-577: https://eclipse-score.github.io/persistency/pr-577/

@arkjedrz
arkjedrz force-pushed the arkjedrz_tooling-06-coverage branch from 3609048 to 7c267c7 Compare October 26, 2026 14:58
Base automatically changed from arkjedrz_tooling-05-lint to main October 26, 2026 15:42
@umaucher
umaucher force-pushed the arkjedrz_tooling-06-coverage branch from 7c267c7 to a639a11 Compare October 26, 2026 15:42
@arkjedrz
arkjedrz force-pushed the arkjedrz_tooling-06-coverage branch from a639a11 to da748d2 Compare October 27, 2026 10:05
- Move quality/coverage to tools/coverage, as in baselibs. Rewrite
  the //quality/coverage labels, the .bazelrc import, the comments
  in MODULE.bazel and BUILD, and the justifications path in
  coverage.yml and release.yml.
- Align the llvm_cov config with baselibs. Set the platform and the
  LLVM and Ferrocene toolchains explicitly. Keep the warning
  feature disables and --instrumentation_filter=^//score[/:].
- Add baselibs' check_coverage_scope.sh and scope_exclusions.txt.
  The check fails when a production library under //score/... is
  neither in //tools/coverage:coverage_scope nor excluded.
- Replace coverage.yml with coverage_report.yml. Use baselibs'
  triggers (push, merge_group, workflow_call) and steps, including
  the scope check, on the cicd common steps with SHA pins. Keep
  workflow_dispatch and the <repo>_coverage_report artifact name.
- Drop the cargo-coverage job from check.yml. coverage_report.yml
  covers rust_kvs. cpp-bazel stays until check.yml is replaced.
- Document the scope check and coverage_report.yml in
  tools/coverage/README.md.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@arkjedrz
arkjedrz force-pushed the arkjedrz_tooling-06-coverage branch from da748d2 to bc4e3d5 Compare October 28, 2026 06:48

This branch has not been deployed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant