Skip to content

[PWGEM] add photon skimmer in photonbuilder #33352

[PWGEM] add photon skimmer in photonbuilder

[PWGEM] add photon skimmer in photonbuilder #33352

Triggered via pull request August 25, 2026 14:14
@SteffimroSteffimro
synchronize #17591
Status Failure
Total duration 22s
Artifacts

o2-linter.yml

on: pull_request_target
O2 linter
18s
O2 linter
Fit to window
Zoom out
Zoom in

Annotations

10 errors, 1 warning, and 2 notices
[name/configurable]: PWGEM/PhotonMeson/TableProducer/photonconversionbuilder.cxx#L194
Use lowerCamelCase for names of configurables and use the same name for the struct member as for the JSON string. (Declare the type and names on the same line.)
[name/configurable]: PWGEM/PhotonMeson/TableProducer/photonconversionbuilder.cxx#L170
Use lowerCamelCase for names of configurables and use the same name for the struct member as for the JSON string. (Declare the type and names on the same line.)
[name/configurable]: PWGEM/PhotonMeson/TableProducer/photonconversionbuilder.cxx#L165
Use lowerCamelCase for names of configurables and use the same name for the struct member as for the JSON string. (Declare the type and names on the same line.)
[name/workflow-file]: PWGEM/PhotonMeson/TableProducer/photonconversionbuilder.cxx#L1
Name of a workflow file must match the name of the main struct in it (without the PWG prefix). (Class implementation files should be in "Core" directories.)
[name/o2-task]: PWGEM/PhotonMeson/TableProducer/photonconversionbuilder.cxx#L1
Specify task name only when it cannot be derived from the struct name. Only append to the default name.
[name/o2-task]: PWGEM/PhotonMeson/TableProducer/photonconversionbuilder.cxx#L1789
Specified task name photon-conversion-builder and the struct name PhotonConversionBuilder produce the same device name photon-conversion-builder. TaskName is redundant.
[name/workflow-file]: PWGEM/Dilepton/TableProducer/filterEoI.cxx#L1
Name of a workflow file must match the name of the main struct in it (without the PWG prefix). (Class implementation files should be in "Core" directories.)
[name/o2-task]: PWGEM/Dilepton/TableProducer/filterEoI.cxx#L1
Specify task name only when it cannot be derived from the struct name. Only append to the default name.
[name/o2-task]: PWGEM/Dilepton/TableProducer/filterEoI.cxx#L195
Device names filter-eoi and filter-eo-i generated from the specified task name filter-eoi and from the struct name filterEoI, respectively, differ in hyphenation. Consider fixing capitalisation of the struct name to FilterEoi and removing TaskName.
[magic-number]: PWGEM/Dilepton/TableProducer/filterEoI.cxx#L118
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.