[PWGEM] SkimmerForDalitzEE: Fix of MegaLinter and CodeCheck - #17337
Merged
Annotations
10 errors, 1 warning, and 2 notices
|
Run tests:
PWGEM/PhotonMeson/TableProducer/skimmerPrimaryElectronFromDalitzEE.cxx#L1
Specify task name only when it cannot be derived from the struct name. Only append to the default name.
|
|
Run tests:
PWGEM/PhotonMeson/TableProducer/skimmerPrimaryElectronFromDalitzEE.cxx#L907
Device names skimmer-primary-electron-from-dalitzee and skimmer-primary-electron-from-dalitz-e-e generated from the specified task name skimmer-primary-electron-from-dalitzee and from the struct name skimmerPrimaryElectronFromDalitzEE, respectively, differ in hyphenation. Consider fixing capitalisation of the struct name to SkimmerPrimaryElectronFromDalitzee and removing TaskName.
|
|
Run tests:
PWGEM/PhotonMeson/TableProducer/skimmerPrimaryElectronFromDalitzEE.cxx#L82
Use UpperCamelCase for names of O2 columns and matching lowerCamelCase names for their getters.
|
|
Run tests:
PWGEM/PhotonMeson/TableProducer/skimmerPrimaryElectronFromDalitzEE.cxx#L872
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
Run tests:
PWGEM/PhotonMeson/TableProducer/skimmerPrimaryElectronFromDalitzEE.cxx#L749
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
Run tests:
PWGEM/PhotonMeson/TableProducer/skimmerPrimaryElectronFromDalitzEE.cxx#L471
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
Run tests:
PWGEM/PhotonMeson/TableProducer/skimmerPrimaryElectronFromDalitzEE.cxx#L443
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
Run tests:
PWGEM/PhotonMeson/TableProducer/skimmerPrimaryElectronFromDalitzEE.cxx#L357
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
Run tests:
PWGEM/PhotonMeson/TableProducer/skimmerPrimaryElectronFromDalitzEE.cxx#L597
Use constant references for non-modified iterators in range-based for loops.
|
|
Run tests:
PWGEM/PhotonMeson/TableProducer/skimmerPrimaryElectronFromDalitzEE.cxx#L357
Avoid hard-coded PDG codes. Use named values from PDG_t or o2::constants::physics::Pdg instead.
|
|
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