Skip to content

Run clang format on code base#339

Open
paulquiring wants to merge 5 commits into
eclipse-score:mainfrom
etas-contrib:feature/run-clang-format-on-code-base
Open

Run clang format on code base#339
paulquiring wants to merge 5 commits into
eclipse-score:mainfrom
etas-contrib:feature/run-clang-format-on-code-base

Conversation

@paulquiring

Copy link
Copy Markdown
Contributor
  • All Cpp files are formatted
  • Enables clang-format pre-commit hook

@github-actions

github-actions Bot commented Jul 21, 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: edb994c7-cb7b-4469-a6d8-3b80fdcd2146
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 (46 packages loaded, 10 targets configured)

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

Analyzing: target //:license-check (107 packages loaded, 57 targets configured)

Analyzing: target //:license-check (158 packages loaded, 3798 targets configured)

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

Analyzing: target //:license-check (170 packages loaded, 9421 targets configured)

Analyzing: target //:license-check (170 packages loaded, 9421 targets configured)

Analyzing: target //:license-check (173 packages loaded, 11309 targets configured)

Analyzing: target //:license-check (173 packages loaded, 11309 targets configured)

Analyzing: target //:license-check (173 packages loaded, 11309 targets configured)

Analyzing: target //:license-check (174 packages loaded, 11433 targets configured)

INFO: Analyzed target //:license-check (175 packages loaded, 11559 targets configured).
[6 / 14] checking cached actions
[12 / 16] JavaToolchainCompileClasses external/rules_java+/toolchains/platformclasspath_classes; 0s disk-cache, processwrapper-sandbox ... (2 actions running)
[15 / 16] Building license.check.license_check.jar (); 0s disk-cache, multiplex-worker
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: 28.359s, Critical Path: 2.08s
INFO: 16 processes: 12 internal, 3 processwrapper-sandbox, 1 worker.
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

@anmittag anmittag moved this from Backlog to Ready in LCM - Lifecycle & Health FT Jul 21, 2026
@paulquiring

Copy link
Copy Markdown
Contributor Author

ensure that formatting-check / format-check also runs //:format.check_C++_with_clang-format

@paulquiring

Copy link
Copy Markdown
Contributor Author

ensure that formatting-check / format-check also runs //:format.check_C++_with_clang-format

is checked now:
//:format.check_C++_with_clang-format PASSED in 1.9s

@paulquiring
paulquiring force-pushed the feature/run-clang-format-on-code-base branch from e7370e7 to 5392d4a Compare July 22, 2026 14:45
@paulquiring
paulquiring force-pushed the feature/run-clang-format-on-code-base branch from 5392d4a to d90d74e Compare July 22, 2026 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Ready

Development

Successfully merging this pull request may close these issues.

2 participants