O2 linter

[Common] Further centrality resolution study histograms added #16136

Sign in to view logs

Triggered via pull request August 25, 2025 12:44

ddobrigk

synchronize #12727

Status Failure

Total duration 15s

Artifacts –

o2-linter.yml

on: pull_request_target

Annotations

10 errors and 2 notices

[name/macro]: Common/Tasks/centralityStudy.cxx#L40

Use SCREAMING_SNAKE_CASE for names of macros. Leading and double underscores are not allowed.

[doc/file]: Common/Tasks/centralityStudy.cxx#L1

Provide mandatory file documentation.

[doc/file]: Common/Tasks/centralityStudy.cxx#L11

Documentation for \author is missing, incorrect or misplaced.

[doc/file]: Common/Tasks/centralityStudy.cxx#L11

Documentation for \brief is missing, incorrect or misplaced.

[doc/file]: Common/Tasks/centralityStudy.cxx#L11

Documentation for \file is missing, incorrect or misplaced.

[magic-number]: Common/Tasks/centralityStudy.cxx#L811

Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.

[magic-number]: Common/Tasks/centralityStudy.cxx#L515

Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.

[root/entity]: Common/Tasks/centralityStudy.cxx#L814

Replace ROOT entities with equivalents from standard C++ or from O2.

[root/entity]: Common/Tasks/centralityStudy.cxx#L566

Replace ROOT entities with equivalents from standard C++ or from O2.

[root/entity]: Common/Tasks/centralityStudy.cxx#L515

Replace ROOT entities with equivalents from standard C++ or from O2.

To tolerate certain issues in a directory, add a line with the test name in "o2linter_config".

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.