[PWGDQ] Please consider the following formatting changes to #10510 #17

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed

alibuild wants to merge 1 commit into iarsene:master from alibuild:alibot-cleanup-10510

Closed

[PWGDQ] Please consider the following formatting changes to #10510 #17

Please consider the following formatting changes

Annotations

2 errors, 10 warnings, and 1 notice

Process completed with exit code 1.

Issues have been found.

PWGDQ/Core/MCSignal.h#L11

Documentation for \brief is missing, incorrect or misplaced.

PWGDQ/Core/MCSignal.h#L11

Documentation for \file is missing, incorrect or misplaced.

PWGDQ/Core/MCSignal.h#L332

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

PWGDQ/Core/MCSignal.h#L284

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

PWGDQ/Core/MCSignal.h#L204

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

PWGDQ/Core/MCSignal.h#L48

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

PWGDQ/Core/MCSignal.h#L43

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

PWGDQ/Core/MCSignal.h#L50

Use O2 logging (LOG, LOGF, LOGP).

PWGDQ/Core/MCSignal.h#L45

Use O2 logging (LOG, LOGF, LOGP).

PWGDQ/Core/MCSignal.h#L63

Do not include iostream. Use O2 logging instead.

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.

The logs for this run have expired and are no longer available.