Skip to content

Add pre-commit config#313

Open
paulquiring wants to merge 4 commits into
eclipse-score:mainfrom
etas-contrib:feature/add-pre-commit-config
Open

Add pre-commit config#313
paulquiring wants to merge 4 commits into
eclipse-score:mainfrom
etas-contrib:feature/add-pre-commit-config

Conversation

@paulquiring

@paulquiring paulquiring commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Uses score_bazel_tools_cc's clang_format_config and quality_clang_tidy_config
Update format workflow

@github-actions

github-actions Bot commented Jul 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: ddd1b643-50a1-4c35-acdb-6042624206d8
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: 
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)

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

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

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

Analyzing: target //:license-check (114 packages loaded, 281 targets configured)

Analyzing: target //:license-check (156 packages loaded, 3003 targets configured)

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

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

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

Analyzing: target //:license-check (166 packages loaded, 7440 targets configured)

Analyzing: target //:license-check (166 packages loaded, 7441 targets configured)

Analyzing: target //:license-check (166 packages loaded, 7441 targets configured)

Analyzing: target //:license-check (168 packages loaded, 10262 targets configured)

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

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

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

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

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

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

INFO: Analyzed target //:license-check (172 packages loaded, 11914 targets configured).
[11 / 16] Creating runfiles tree bazel-out/k8-opt-exec-ST-d57f47055a04/bin/external/score_tooling+/dash/tool/formatters/dash_format_converter.runfiles [for tool]; 0s local
[14 / 16] JavaToolchainCompileBootClasspath external/rules_java+/toolchains/platformclasspath.jar; 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: 117.032s, Critical Path: 2.52s
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>]

@paulquiring paulquiring changed the title add pre commit config Add pre-commit config Jul 14, 2026
@github-actions

Copy link
Copy Markdown

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

* SPDX-License-Identifier: Apache-2.0
********************************************************************************/

#ifndef IPC_DROPIN_RINGBUFFER_HPP_

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.

Suggested change
#ifndef IPC_DROPIN_RINGBUFFER_HPP_
#ifndef IPC_DROPIN_RINGBUFFER_HPP_

* SPDX-License-Identifier: Apache-2.0
********************************************************************************/

#ifndef IPC_DROPIN_SOCKET_HPP_

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.

Suggested change
#ifndef IPC_DROPIN_SOCKET_HPP_
#ifndef IPC_DROPIN_SOCKET_HPP_

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