[PATCH] D56424: [clang-tidy] Add check for underscores in googletest names.

Kar Epker via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 8 13:34:25 PST 2019


karepker added inline comments.


================
Comment at: docs/ReleaseNotes.rst:161
+
+  Checks that Googletest test and test case names do not contain an underscore,
+  which is prohibited by the Googletest FAQ.
----------------
Eugene.Zelenko wrote:
> Please synchronize with first statement in documentation.
Done, I think. I assume by documentation you mean documentation in the check rst file. The first statement there isn't really a sentence, but I took as much of the wording from there as I could.


Repository:
  rCTE Clang Tools Extra

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56424/new/

https://reviews.llvm.org/D56424





More information about the cfe-commits mailing list