Skip to content
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

fix(ndt_omp): update clang-format and format code #58

Merged
merged 2 commits into from
Jul 19, 2024

Conversation

a-maumau
Copy link

@a-maumau a-maumau commented Jul 19, 2024

Update the .clang-format to synchronize with autoware.universe, and apply clang-format to the codes.
Some of the header was not possible to reorder, it will be ignore with // clang-format off.

Checked with compare_regression_test_result.py, and confirmed the test is passing.

(copy of #56 which was not able to pass the checks because of some error on fetching the branch from forked repo.)

a-maumau added 2 commits July 12, 2024 14:43
Signed-off-by: a-maumau <maumaumaumaumaumaumaumaumaumau@gmail.com>
Signed-off-by: a-maumau <maumaumaumaumaumaumaumaumaumau@gmail.com>
Copy link

@SakodaShintaro SakodaShintaro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the late review
Looks Good To Me

@SakodaShintaro SakodaShintaro merged commit c482edd into tier4/main Jul 19, 2024
8 of 10 checks passed
@SakodaShintaro SakodaShintaro deleted the mau/fix/clang-format branch July 19, 2024 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants