Please consider the following formatting changes to #10025 #15

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.

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

Merged

Feb 17, 2025

Merged

Please consider the following formatting changes

Annotations

2 errors, 10 warnings, and 1 notice

Issues have been found.

Process completed with exit code 1.

Including iostream is discouraged. Use O2 logging instead.

Consider using the PI constant (and its multiples and fractions) defined in o2::constants::math.

Consider using the PI constant (and its multiples and fractions) defined in o2::constants::math.

Consider using the PI constant (and its multiples and fractions) defined in o2::constants::math.

Consider using the PI constant (and its multiples and fractions) defined in o2::constants::math.

Documentation for \file is missing, incorrect or misplaced.

Documentation for \brief is missing, incorrect or misplaced.

Documentation for \author is missing, incorrect or misplaced.

Provide mandatory file documentation.

Use lowerCamelCase for names of functions and variables.

You can disable a test for a line by adding a comment with "o2-linter: disable=" followed by the name of the test.