[Common] Please consider the following formatting changes to #14114 #64

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

romainschotter merged 1 commit into romainschotter:master from alibuild:alibot-cleanup-14114

Dec 12, 2025

Merged

[Common] Please consider the following formatting changes to #14114 #64

Please consider the following formatting changes

Annotations

4 errors and 2 notices

Process completed with exit code 1.

Issues have been found.

Common/Tools/TrackPropagationModule.h#L74

Use lowerCamelCase for names of configurables and use the same name for the struct member as for the JSON string. (Declare the type and names on the same line.)

Common/Tools/TrackPropagationModule.h#L179

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

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.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Common] Please consider the following formatting changes to #14114 #64

Uh oh!

Uh oh!

[Common] Please consider the following formatting changes to #14114 #64

Uh oh!

Annotations

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

Re-running jobs...