[PWGDQ] clang-tidy bug fixes in tableMaker(MC)_withAssoc#16909
Open
mcoquet642 wants to merge 24 commits into
Open
[PWGDQ] clang-tidy bug fixes in tableMaker(MC)_withAssoc#16909mcoquet642 wants to merge 24 commits into
mcoquet642 wants to merge 24 commits into
Annotations
10 errors, 1 warning, and 2 notices
|
Run tests:
PWGDQ/TableProducer/tableMakerMC_withAssoc.cxx#L1117
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
Run tests:
PWGDQ/TableProducer/tableMakerMC_withAssoc.cxx#L1070
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
Run tests:
PWGDQ/TableProducer/tableMakerMC_withAssoc.cxx#L1048
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
Run tests:
PWGDQ/TableProducer/tableMakerMC_withAssoc.cxx#L899
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
Run tests:
PWGDQ/TableProducer/tableMakerMC_withAssoc.cxx#L603
Use O2 logging (LOG, LOGF, LOGP).
|
|
Run tests:
PWGDQ/TableProducer/tableMakerMC_withAssoc.cxx#L593
Use O2 logging (LOG, LOGF, LOGP).
|
|
Run tests:
PWGDQ/TableProducer/tableMakerMC_withAssoc.cxx#L588
Use O2 logging (LOG, LOGF, LOGP).
|
|
Run tests:
PWGDQ/TableProducer/tableMakerMC_withAssoc.cxx#L586
Use O2 logging (LOG, LOGF, LOGP).
|
|
Run tests:
PWGDQ/TableProducer/tableMakerMC_withAssoc.cxx#L583
Use O2 logging (LOG, LOGF, LOGP).
|
|
Run tests:
PWGDQ/TableProducer/tableMakerMC_withAssoc.cxx#L147
Use O2 logging (LOG, LOGF, LOGP).
|
|
Complete job
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/
|
|
Run tests
To tolerate certain issues in a directory, add a line with the test name in "o2linter_config".
|
|
Run tests
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.
|
background
wait
wait-all
cancel
parallel
Loading