Skip to content

[Common] Use const& and std::move to avoid copies #33734

[Common] Use const& and std::move to avoid copies

[Common] Use const& and std::move to avoid copies #33734

Triggered via pull request September 1, 2026 15:43
@vkuceravkucera
opened #17703
Status Failure
Total duration 18s
Artifacts

o2-linter.yml

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

Annotations

10 errors, 1 warning, and 2 notices
[magic-number]: Common/Core/EventPlaneHelper.cxx#L142
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
[magic-number]: Common/Core/EventPlaneHelper.cxx#L139
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
[magic-number]: Common/Core/EventPlaneHelper.cxx#L114
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
[magic-number]: Common/Core/EventPlaneHelper.cxx#L102
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
[magic-number]: Common/Core/EventPlaneHelper.cxx#L72
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
[logging]: Common/Core/EventPlaneHelper.cxx#L103
Use O2 logging (LOG, LOGF, LOGP).
[logging]: Common/Core/EventPlaneHelper.cxx#L96
Use O2 logging (LOG, LOGF, LOGP).
[magic-number]: Common/CCDB/macros/ctpRateF.C#L141
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
[magic-number]: Common/CCDB/macros/ctpRateF.C#L131
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
[magic-number]: Common/CCDB/macros/ctpRateF.C#L100
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.