[PWGDQ] Clang-tidy fixes in HistogramManager and HistogramsLibrary #31149
Triggered via pull request
July 13, 2026 15:05
mcoquet642
mcoquet642
opened
#17047
Status
Failure
Total duration
16s
Artifacts
–
Annotations
10 errors, 1 warning, and 2 notices
|
[const-ref-in-for-loop]:
PWGDQ/Core/HistogramsLibrary.cxx#L2773
Use constant references for non-modified iterators in range-based for loops.
|
|
[const-ref-in-for-loop]:
PWGDQ/Core/HistogramsLibrary.cxx#L2759
Use constant references for non-modified iterators in range-based for loops.
|
|
[const-ref-in-for-loop]:
PWGDQ/Core/HistogramsLibrary.cxx#L2748
Use constant references for non-modified iterators in range-based for loops.
|
|
[const-ref-in-for-loop]:
PWGDQ/Core/HistogramsLibrary.cxx#L2685
Use constant references for non-modified iterators in range-based for loops.
|
|
[const-ref-in-for-loop]:
PWGDQ/Core/HistogramsLibrary.cxx#L2530
Use constant references for non-modified iterators in range-based for loops.
|
|
[logging]:
PWGDQ/Core/HistogramManager.cxx#L1007
Use O2 logging (LOG, LOGF, LOGP).
|
|
[logging]:
PWGDQ/Core/HistogramManager.cxx#L1004
Use O2 logging (LOG, LOGF, LOGP).
|
|
[logging]:
PWGDQ/Core/HistogramManager.cxx#L998
Use O2 logging (LOG, LOGF, LOGP).
|
|
[logging]:
PWGDQ/Core/HistogramManager.cxx#L997
Use O2 logging (LOG, LOGF, LOGP).
|
|
[include-iostream]:
PWGDQ/Core/HistogramManager.cxx#L30
Do not include iostream. Use O2 logging instead.
|
|
O2 linter
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: thollander/actions-comment-pull-request@v3. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
O2 linter
To tolerate certain issues in a directory, add a line with the test name in "o2linter_config".
|
|
O2 linter
Exceptionally, you can disable a test for a line by adding a comment with "o2-linter: disable=" followed by the name of the test and parentheses with a reason for the exception.
|