[PWGDQ] some clang-tidy fixes for VarManager.h #30902
Triggered via pull request
July 8, 2026 21:17
mcoquet642
mcoquet642
synchronize
#16962
Status
Failure
Total duration
20s
Artifacts
–
Annotations
10 errors, 1 warning, and 2 notices
|
[magic-number]:
PWGDQ/Core/VarManager.h#L1827
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
PWGDQ/Core/VarManager.h#L1765
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
PWGDQ/Core/VarManager.h#L1740
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
PWGDQ/Core/VarManager.h#L1513
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
PWGDQ/Core/VarManager.cxx#L461
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
PWGDQ/Core/VarManager.cxx#L294
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[const-ref-in-for-loop]:
PWGDQ/Core/VarManager.cxx#L507
Use constant references for non-modified iterators in range-based for loops.
|
|
[const-ref-in-for-loop]:
PWGDQ/Core/VarManager.cxx#L468
Use constant references for non-modified iterators in range-based for loops.
|
|
[std-prefix]:
PWGDQ/Core/VarManager.cxx#L195
Use std:: prefix for names from the std namespace.
|
|
[std-prefix]:
PWGDQ/Core/VarManager.cxx#L194
Use std:: prefix for names from the std namespace.
|
|
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.
|