O2 linter

Please consider the following formatting changes to #11866 #18

Sign in to view logs

Triggered via pull request July 4, 2025 17:58

alibuild

opened #11

Status Failure

Total duration 16s

Artifacts –

o2-linter.yml

on: pull_request_target

Annotations

10 errors and 2 notices

[magic-number]: PWGDQ/Tasks/TagAndProbe.cxx#L331

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

[magic-number]: PWGDQ/Tasks/TagAndProbe.cxx#L329

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

[magic-number]: PWGDQ/Tasks/TagAndProbe.cxx#L328

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

[const-ref-in-process]: PWGDQ/Tasks/TagAndProbe.cxx#L1

Use constant references for table subscriptions in process functions.

[const-ref-in-process]: PWGDQ/Tasks/TagAndProbe.cxx#L352

Argument MyEvents& is not const&.

[const-ref-in-for-loop]: PWGDQ/Tasks/TagAndProbe.cxx#L310

Use constant references for non-modified iterators in range-based for loops.

[const-ref-in-for-loop]: PWGDQ/Tasks/TagAndProbe.cxx#L299

Use constant references for non-modified iterators in range-based for loops.

[logging]: PWGDQ/Tasks/TagAndProbe.cxx#L104

Use O2 logging (LOG, LOGF, LOGP).

[root/entity]: PWGDQ/Tasks/TagAndProbe.cxx#L375

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

[include-iostream]: PWGDQ/Tasks/TagAndProbe.cxx#L55

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.