[DPG] Please consider the following formatting changes to #13514 #23
alibuildAnnotations
10 errors and 2 notices
|
[magic-number]:
DPG/Tasks/AOTTrack/qaMatchEff.cxx#L1779
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
DPG/Tasks/AOTTrack/qaMatchEff.cxx#L1762
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
DPG/Tasks/AOTTrack/qaMatchEff.cxx#L1745
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
DPG/Tasks/AOTTrack/qaMatchEff.cxx#L1728
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
DPG/Tasks/AOTTrack/qaMatchEff.cxx#L1711
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
DPG/Tasks/AOTTrack/qaMatchEff.cxx#L1694
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
DPG/Tasks/AOTTrack/qaMatchEff.cxx#L1624
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[pdg/explicit-code]:
DPG/Tasks/AOTTrack/qaMatchEff.cxx#L3134
Avoid hard-coded PDG codes. Use named values from PDG_t or o2::constants::physics::Pdg instead.
|
|
[two-pi-add-subtract]:
DPG/Tasks/AOTTrack/qaMatchEff.cxx#L1614
Use RecoDecay::constrainAngle to restrict angle to a given range.
|
|
[two-pi-add-subtract]:
DPG/Tasks/AOTTrack/qaMatchEff.cxx#L1611
Use RecoDecay::constrainAngle to restrict angle to a given range.
|
|
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.
|