Skip to content

[ci] add clang-tidy code analysis - #22664

Open
jblomer wants to merge 9 commits into
root-project:masterfrom
jblomer:ci-tidy
Open

[ci] add clang-tidy code analysis#22664
jblomer wants to merge 9 commits into
root-project:masterfrom
jblomer:ci-tidy

Conversation

@jblomer

@jblomer jblomer commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Currently only checks the tree/ntuple code part. Other parts can plug in themselves by creating a .clang-tidy config file.

@jblomer jblomer self-assigned this Jun 19, 2026
@jblomer
jblomer requested a review from dpiparo as a code owner June 19, 2026 12:05
@jblomer
jblomer marked this pull request as draft June 19, 2026 12:05
@jblomer
jblomer force-pushed the ci-tidy branch 8 times, most recently from 13bc56e to 5fc03e7 Compare June 19, 2026 13:25
@github-actions

github-actions Bot commented Jun 19, 2026

Copy link
Copy Markdown

Test Results

    23 files      23 suites   3d 19h 12m 29s ⏱️
 3 860 tests  3 859 ✅ 0 💤 1 ❌
79 582 runs  79 581 ✅ 0 💤 1 ❌

For more details on these failures, see this check.

Results for commit 9cdd344.

♻️ This comment has been updated with latest results.

@jblomer
jblomer force-pushed the ci-tidy branch 15 times, most recently from c5ef02b to c3dcd87 Compare July 9, 2026 12:44
@jblomer jblomer changed the title [ci] add clang-tidy code analysis (WIP) [ci] add clang-tidy code analysis Jul 9, 2026
@jblomer
jblomer marked this pull request as ready for review July 9, 2026 13:20
@jblomer
jblomer requested a review from silverweed as a code owner July 9, 2026 13:20
@jblomer
jblomer requested a review from pcanal as a code owner July 9, 2026 13:20
Comment thread tree/ntuple/src/RNTupleDescriptor.cxx Outdated
Comment thread tree/ntuple/src/RNTupleSerialize.cxx Outdated
@jblomer
jblomer requested a review from hageboeck September 1, 2026 14:58
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