Skip to content

Latest commit

 

History

History
29 lines (19 loc) · 1.21 KB

clang_tidy.md

File metadata and controls

29 lines (19 loc) · 1.21 KB
Raw

clang-tidy checks

Warning is not useful

If you found that a warning produced by clang-tidy is not useful:

  • If clang-tidy must not run for some files at all (e.g. lit test), please add files to ignorelist.

  • Consider fixing or suppressing diagnostic if there is a good reason.

  • File a bug if build process should be improved.

  • If you believe that you found a clang-tidy bug then please keep in mind that clang-tidy version run by bot might not be most recent. Please reproduce your issue on current version before submitting a bug to clang-tidy.

Review comments

Build bot leaves inline comments only for a small subset of files that are not blacklisted for analysis (see above) and specifically whitelisted for comments.

That is done to avoid potential noise when a file already contains a number of warnings.

If your are confident that some files are in good shape already, please whitelist them.

about pre-merge checks