-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
Treat clang-analyzer-osx warnings as errors #29755
Conversation
@@ -1 +1,2 @@ | |||
Checks: 'google-*' | |||
WarningsAsErrors: 'clang-analyzer-osx.*' |
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.
Does this file allow comments? If so, we should link to a tracking issue.
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.
Per https://clang.llvm.org/extra/clang-tidy/ it looks like this is YAML, so #
should work for comments.
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.
Done.
There are no
clang-analyzer-osx
warnings in the iOS or macOS embedder as of #29623. Make the clang-tidy linter treat these warnings as errors so they show up locally during pre-push git hook linter.Note this still isn't running in CI for due to flutter/flutter#61661 (comment).
Pre-submit part of flutter/flutter#93279
Pre-launch Checklist
writing and running engine tests.
///
).If you need help, consider asking for advice on the #hackers-new channel on Discord.