[Common] Please consider the following formatting changes to #13213 #16

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 amaringarcia:master from alibuild:alibot-cleanup-13213

Closed

[Common] Please consider the following formatting changes to #13213 #16

Please consider the following formatting changes

Annotations

10 errors and 2 notices

Common/Tools/PID/pidTPCModule.h#L706

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

Common/Tools/PID/pidTPCModule.h#L694

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

Common/Tools/PID/pidTPCModule.h#L692

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

Common/Tools/PID/pidTPCModule.h#L681

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

Common/Tools/PID/pidTPCModule.h#L538

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

Common/Tools/PID/pidTPCModule.h#L535

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

Common/Tools/PID/pidTPCModule.h#L456

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

Common/Tools/PID/pidTPCModule.h#L453

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

Common/Tools/PID/pidTPCModule.h#L435

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

Common/Tools/PID/pidTPCModule.h#L136

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

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.

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