Skip to content

Update mock bazel rules to match file names - #475

Open
cameron-craig-etas wants to merge 1 commit into
eclipse-score:mainfrom
etas-contrib:update-mock-names-in-bazel
Open

Update mock bazel rules to match file names#475
cameron-craig-etas wants to merge 1 commit into
eclipse-score:mainfrom
etas-contrib:update-mock-names-in-bazel

Conversation

@cameron-craig-etas

@cameron-craig-etas cameron-craig-etas commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

#422

This PR is a follow-up to /pull/457, where we updated the mocks filenames to match the mock_ naming convention. We're now making sure the BUILD rule names use the same format, to avoid confusion.

@github-actions

github-actions Bot commented Aug 14, 2026

Copy link
Copy Markdown

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.6.0) and connecting to it...
INFO: Invocation ID: ebf9217d-8d21-4f18-8a6a-a0d388b4e92e
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 (36 packages loaded, 10 targets configured)

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

Analyzing: target //:license-check (151 packages loaded, 1550 targets configured)

Analyzing: target //:license-check (159 packages loaded, 6268 targets configured)

Analyzing: target //:license-check (164 packages loaded, 6285 targets configured)

Analyzing: target //:license-check (171 packages loaded, 8415 targets configured)

Analyzing: target //:license-check (176 packages loaded, 11463 targets configured)

Analyzing: target //:license-check (176 packages loaded, 11463 targets configured)

INFO: Analyzed target //:license-check (177 packages loaded, 11589 targets configured).
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: 23.725s, Critical Path: 0.20s
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>]

@github-actions

Copy link
Copy Markdown

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

alias(
name = "report_running_mock_cc",
actual = "//score/launch_manager/src/lifecycle_client:report_running_mock",
name = "mock_report_running_cc",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

These are actually our public targets, I think we can just keep them in the "wrong" format. This is only used by our users so doesn't make sense to change.

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