Skip to content

ci: Add clang tidy and ruff checks and fixes - #283

Merged
lurtz merged 18 commits into
mainfrom
add-clang-tidy-checks
Sep 1, 2026
Merged

lurtz merged 18 commits into
mainfrom
add-clang-tidy-checks

Conversation

@lurtz

@lurtz lurtz commented Aug 31, 2026 •

Copy link
Copy Markdown
Contributor

We intend to move to over to communication. To ease migration lets run the same clang-tidy and ruff checks as they do.

I tried first to use the current S-CORE tooling, but communications setup seems to work better. E.g. it autodiscovers all files and needs no further changes if more C++ / Python code is added.

The C++ changes are almost all autofixes from clang-tidy.

The Python changes are formatting changes due to introducing a .ruff.toml which allows 120 character lines.

@github-actions

github-actions Bot commented Aug 31, 2026 •

Copy link
Copy Markdown
Contributor

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run //: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: bdb63b39-7bb6-4f4e-a8ac-64cee93c6cdd
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
WARNING: For repository 'score_bazel_platforms', the root module requires module version score_bazel_platforms@1.0.0, but got score_bazel_platforms@1.1.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: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 2 packages loaded
WARNING: Target pattern parsing failed.
ERROR: Skipping '//:license-check': no such target '//:license-check': target 'license-check' not declared in package '' defined by /home/runner/work/inc_someip_gateway/inc_someip_gateway/BUILD
ERROR: no such target '//:license-check': target 'license-check' not declared in package '' defined by /home/runner/work/inc_someip_gateway/inc_someip_gateway/BUILD
INFO: Elapsed time: 12.607s
INFO: 0 processes.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target

@github-actions

github-actions Bot commented Aug 31, 2026 •

Copy link
Copy Markdown
Contributor

Coverage report for this pull request:

Status: success (workflow run)
Coverage data: coverage-report

The archive contains the HTML report, the raw LCOV data and the JUnit XML test results.

@lurtz lurtz changed the title Add clang tidy checks ci: Add clang tidy and ruff checks and fixes Aug 31, 2026
@lurtz
lurtz marked this pull request as ready for review August 31, 2026 17:00
@lurtz
lurtz force-pushed the add-clang-tidy-checks branch from d336647 to 0be9ac7 Compare August 31, 2026 17:04
@lurtz
lurtz enabled auto-merge August 31, 2026 18:21
@github-actions

Copy link
Copy Markdown
Contributor

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

Comment thread static_analysis.bazelrc
@lurtz
lurtz added this pull request to the merge queue Sep 1, 2026
Merged via the queue into main with commit d1998e6 Sep 1, 2026
25 checks passed
@lurtz
lurtz deleted the add-clang-tidy-checks branch September 1, 2026 15:01
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.

3 participants