[PATCH] D97753: [clang-tidy] Add a check for enforcing minimum length for variable names

[Florin Iucha via Phabricator via cfe-commits]

0x8000-0000 marked an inline comment as done.
0x8000-0000 added inline comments.


================
Comment at: clang-tools-extra/docs/clang-tidy/checks/readability-variable-length.rst:9
+
+Loop counter variables are expected to have a length of at least
+`MinimumLoopCounterNameLength` characters (default is 2). Additionally, `i`,
----------------
Eugene.Zelenko wrote:
> Eugene.Zelenko wrote:
> > See other checks documentation as example of options descriptions.
> You still need to add `Options` section and mark-up for options. See other checks documentation as example.
Oh, sorry, now I see what you mean. I have sampled randomly a handful of documentation snippets, and didn't find anything. But after your comment, I looked at readability-identifier-naming.rst which had "options" in all their glory.


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

https://reviews.llvm.org/D97753