-
Notifications
You must be signed in to change notification settings - Fork 388
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
iox-#1998 enable bazel build with clang #1999
iox-#1998 enable bazel build with clang #1999
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1999 +/- ##
==========================================
- Coverage 74.32% 74.30% -0.02%
==========================================
Files 414 414
Lines 16086 16086
Branches 2249 2249
==========================================
- Hits 11956 11953 -3
Misses 3416 3416
- Partials 714 717 +3
Flags with carried forward coverage won't be shown. Click here to find out more. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. LGTM
Thanks for your contribution
@phongt please change |
Done. |
Pre-Review Checklist for the PR Author
iox-123-this-is-a-branch
)iox-#123 commit text
)task-list-completed
)iceoryx_hoofs
are added to./clang-tidy-diff-scans.txt
Notes for Reviewer
Checklist for the PR Reviewer
iceoryx_hoofs
have been added to./clang-tidy-diff-scans.txt
Post-review Checklist for the PR Author
References