[PATCH] D126891: [clang-tidy] The check should ignore final classes

[Balázs Benics via Phabricator via cfe-commits]

steakhal added inline comments.


================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:170
+  <clang-tidy/checks/cppcoreguidelines-virtual-class-destructor>` involving
+  `final` classes. The check will not diagnose `final` marked classes, since
+  those cannot be used as base classes, consequently they can not violate the
----------------
carlosgalvezp wrote:
> I believe the convention is to use double-backtick for C++ keywords and functions, see line 160.
Thanks, I'll fix it. I should learn rst for once.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126891