[PATCH] D52984: [analyzer] Checker reviewer's checklist
Umann Kristóf via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sat Nov 10 07:03:28 PST 2018
Szelethus added inline comments.
Comment at: www/analyzer/checker_dev_manual.html:719
+<li>User facing documentation is important for adoption! Make sure the check list updated
+ at the homepage of the analyzer. Also ensure that the description is good quality in
> Szelethus wrote:
> > Make sure the **checker** list **is** updated
> I think at some point we should decide if we prefer the term check or checker to refer to these things :) Clang Tidy clearly prefers check.
That is the distinction I'm aware of too: checkers in the Static Analyzer, checks in clang-tidy.
More information about the cfe-commits