revi-archive/llvm-premerge-checks
1
0
Fork 0
Code Activity
llvm-premerge-checks/docs/clang_tidy.md
Mikhail Goncharov a3f6610df2 Update bot report 
- add links to join beta and report issue
- add link "not useful" to clang-tidy warning
- clang-tidy comment in report now tells how many inline comments were
  added
2020-01-23 13:35:40 +01:00

1.2 KiB
Raw Permalink Blame History

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 blacklist.

  • 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