O2 linter

[PWGDQ] Please consider the following formatting changes to #12679 #71

Sign in to view logs

Triggered via pull request August 21, 2025 16:27

alibuild

opened #25

Status Failure

Total duration 18s

Artifacts –

o2-linter.yml

on: pull_request_target

Annotations

10 errors and 2 notices

[doc/file]: PWGDQ/Core/VarManager.cxx#L1

Provide mandatory file documentation.

[doc/file]: PWGDQ/Core/VarManager.cxx#L11

Documentation for \author is missing, incorrect or misplaced.

[doc/file]: PWGDQ/Core/VarManager.cxx#L11

Documentation for \brief is missing, incorrect or misplaced.

[doc/file]: PWGDQ/Core/VarManager.cxx#L11

Documentation for \file is missing, incorrect or misplaced.

[magic-number]: PWGDQ/Core/VarManager.cxx#L267

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

[root/entity]: PWGDQ/Core/VarManager.cxx#L362

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

[root/entity]: PWGDQ/Core/VarManager.cxx#L108

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

[std-prefix]: PWGDQ/Core/VarManager.cxx#L168

Use std:: prefix for names from the std namespace.

[std-prefix]: PWGDQ/Core/VarManager.cxx#L167

Use std:: prefix for names from the std namespace.

[include-iostream]: PWGDQ/Core/VarManager.cxx#L12

Do not include iostream. Use O2 logging instead.

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.