[PATCH] D157390: [clang-tidy][include-cleaner] Add option to control deduplication of findings per symbol

[Piotr Zegar via Phabricator via cfe-commits]

PiotrZSL added inline comments.


================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:195
 
+- Misc-include-cleaner check has option `DeduplicateFindings` to output one
+  finding per occurence of a symbol.
----------------
invalid release note should be something like:
```
Improved :doc:`misc-include-cleaner <clang-tidy/checks/misc/include-cleaner>` check by adding option `DeduplicateFindings` to output one finding per symbol occurence.
```

Also this list is sorted per check, so it should be before modernize-loop-convert.


================
Comment at: clang-tools-extra/docs/clang-tidy/checks/misc/include-cleaner.rst:40
+   A boolean that controls whether the check should deduplicate findings for the
+   same symbol. Defaults to true.
----------------
true -> `true`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157390