Skip to content

enable clang-format checker CI job - #18

Open
MonikaKumari26 wants to merge 1 commit into
eclipse-score:mainfrom
MonikaKumari26:enable_clang_check_CI_Job
Open

enable clang-format checker CI job#18
MonikaKumari26 wants to merge 1 commit into
eclipse-score:mainfrom
MonikaKumari26:enable_clang_check_CI_Job

Conversation

@MonikaKumari26

@MonikaKumari26 MonikaKumari26 commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Enable clang format checker for CI

  • .clang-format : C++ formatting rules
  • .github/workflows/format.yml : CI job: runs bazel format check on every PR
  • .pre-commit-config.yaml : pre-commit hook: clang-format only

@github-actions

github-actions Bot commented Aug 3, 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]
2026/08/03 13:21:52 Downloading https://releases.bazel.build/8.6.0/release/bazel-8.6.0-linux-x86_64...
Extracting Bazel installation...
Starting local Bazel server (8.6.0) and connecting to it...
INFO: Invocation ID: 8b85a11f-f015-4965-8c72-4d30d870b52f
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
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: 
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 (15 packages loaded, 14 targets configured)

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

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

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

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

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

Analyzing: target //:license-check (147 packages loaded, 2423 targets configured)

Analyzing: target //:license-check (161 packages loaded, 6181 targets configured)

Analyzing: target //:license-check (163 packages loaded, 9569 targets configured)

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

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

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

Analyzing: target //:license-check (165 packages loaded, 9694 targets configured)

Analyzing: target //:license-check (172 packages loaded, 13612 targets configured)

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

INFO: Analyzed target //:license-check (175 packages loaded, 15822 targets configured).
INFO: From Generating Dash formatted dependency file ...:
INFO: Successfully converted 2 packages from Cargo.lock to bazel-out/k8-fastbuild/bin/formatted.txt
[13 / 16] JavaToolchainCompileClasses external/rules_java+/toolchains/platformclasspath_classes; 0s disk-cache, processwrapper-sandbox
[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: 69.921s, Critical Path: 2.53s
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

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown

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

@MonikaKumari26
MonikaKumari26 requested a review from stmuench August 3, 2026 12:34
@MonikaKumari26
MonikaKumari26 force-pushed the enable_clang_check_CI_Job branch from 509e5d6 to 16eec41 Compare August 3, 2026 13:01
@MonikaKumari26 MonikaKumari26 changed the title enable clang-tidy and clang-format checker CI job enable clang-format checker CI job Aug 3, 2026
@MonikaKumari26
MonikaKumari26 force-pushed the enable_clang_check_CI_Job branch from 16eec41 to d3bd6b4 Compare August 3, 2026 13:21
Comment thread MODULE.bazel
Comment on lines +140 to +149
bazel_dep(name = "score_bazel_tools_cc", version = "0.1.0", dev_dependency = True)
bazel_dep(name = "buildifier_prebuilt", version = "8.5.1.2", dev_dependency = True)
bazel_dep(name = "toolchains_llvm", version = "1.4.0", dev_dependency = True)

llvm = use_extension("@toolchains_llvm//toolchain/extensions:llvm.bzl", "llvm", dev_dependency = True)
llvm.toolchain(
llvm_version = "19.1.1",
stdlib = {"linux-x86_64": "stdc++"},
)
use_repo(llvm, "llvm_toolchain")

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.

why are these required? please remove if they aren't

Comment thread MODULE.bazel
"python",
dev_dependency = True,
)
python.single_version_override(

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.

where is such override actually needed? can we somehow avoid it?

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.

2 participants