[PWGEM] Clean up of clang-tidy errors and warnings from #17288 #32236
Triggered via pull request
August 4, 2026 11:34
mhemmer-cern
synchronize
#17310
Status
Failure
Total duration
23s
Artifacts
–
Annotations
10 errors, 1 warning, and 2 notices
|
[name/o2-task]:
PWGEM/PhotonMeson/TableProducer/photonconversionbuilder.cxx#L1126
Specified task name photon-conversion-builder and the struct name PhotonConversionBuilder produce the same device name photon-conversion-builder. TaskName is redundant.
|
|
[magic-number]:
PWGEM/PhotonMeson/TableProducer/photonconversionbuilder.cxx#L1061
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
PWGEM/PhotonMeson/TableProducer/photonconversionbuilder.cxx#L845
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
PWGEM/PhotonMeson/TableProducer/photonconversionbuilder.cxx#L709
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
PWGEM/PhotonMeson/TableProducer/photonconversionbuilder.cxx#L696
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
PWGEM/PhotonMeson/TableProducer/photonconversionbuilder.cxx#L652
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
PWGEM/PhotonMeson/TableProducer/photonconversionbuilder.cxx#L649
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
PWGEM/PhotonMeson/TableProducer/photonconversionbuilder.cxx#L389
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
PWGEM/PhotonMeson/TableProducer/photonconversionbuilder.cxx#L358
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
PWGEM/PhotonMeson/TableProducer/photonconversionbuilder.cxx#L355
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
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.
|