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

ci(clang-tidy): enable bugprone checkings #5655

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

veqcc
Copy link
Contributor

@veqcc veqcc commented Jan 15, 2025

Description

This is a subsequent PR of #5543.

Here, the following checks are newly enabled:

  • bugprone-branch-clone
  • bugprone-exception-escape
  • bugprone-integer-division
  • bugprone-macro-parentheses
  • bugprone-parent-virtual-call
  • bugprone-reserved-identifier
  • bugprone-signed-char-misuse

The warnings of these checks have been removed by the following PRs:

autowarefoundation/autoware.universe#9627
autowarefoundation/autoware.universe#9628
autowarefoundation/autoware.universe#9629
autowarefoundation/autoware.universe#9630
autowarefoundation/autoware.universe#9631
autowarefoundation/autoware.universe#9643
autowarefoundation/autoware.universe#9647
autowarefoundation/autoware.universe#9648
autowarefoundation/autoware.universe#9651
autowarefoundation/autoware.universe#9652
autowarefoundation/autoware.universe#9654
autowarefoundation/autoware.universe#9658
autowarefoundation/autoware.universe#9659
autowarefoundation/autoware.universe#9660
autowarefoundation/autoware.universe#9662
autowarefoundation/autoware.universe#9668
autowarefoundation/autoware.universe#9669
autowarefoundation/autoware.universe#9670
autowarefoundation/autoware.universe#9671
autowarefoundation/autoware.universe#9697
autowarefoundation/autoware.universe#9698
autowarefoundation/autoware.universe#9699
autowarefoundation/autoware.universe#9700
autowarefoundation/autoware.universe#9701
autowarefoundation/autoware.universe#9702
autowarefoundation/autoware.universe#9712
autowarefoundation/autoware.universe#9715
autowarefoundation/autoware.universe#9723
autowarefoundation/autoware.universe#9724
autowarefoundation/autoware.universe#9725
autowarefoundation/autoware.universe#9726
autowarefoundation/autoware.universe#9727
autowarefoundation/autoware.universe#9729
autowarefoundation/autoware.universe#9730
autowarefoundation/autoware.universe#9732
autowarefoundation/autoware.universe#9738
autowarefoundation/autoware.universe#9780
autowarefoundation/autoware.universe#9781
autowarefoundation/autoware.universe#9805
autowarefoundation/autoware.universe#9815

How was this PR tested?

Notes for reviewers

You can verify it locally by the similar configurations with #5543 (comment)

Effects on system behavior

None.

@veqcc veqcc self-assigned this Jan 15, 2025
Copy link

Thank you for contributing to the Autoware project!

🚧 If your pull request is in progress, switch it to draft mode.

Please ensure:

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.

1 participant