ishaangandhi added inline comments.
================ Comment at: clang-tools-extra/docs/ReleaseNotes.rst:236 +- Fixed a false positive in :doc:`bugprone-branch-clone + <clang-tidy/checks/bugprone-branch-clone>` when the branches ---------------- Eugene.Zelenko wrote: > ishaangandhi wrote: > > Eugene.Zelenko wrote: > > > ishaangandhi wrote: > > > > Eugene.Zelenko wrote: > > > > > Eugene.Zelenko wrote: > > > > > > ishaangandhi wrote: > > > > > > > ishaangandhi wrote: > > > > > > > > Eugene.Zelenko wrote: > > > > > > > > > Documentation path was changed recently. Please also keep > > > > > > > > > alphabetical order inside section. > > > > > > > > Hm? Its still at this path on mastesr: > > > > > > > > https://github.com/llvm/llvm-project/blob/main/clang-tools-extra/docs/ReleaseNotes.rst > > > > > > > Also, it doesn't look alphabetized to me? Two points up is > > > > > > > "performance-unnecessary-..." then its "bugprone-use-after...". > > > > > > Sorry, my mistake. > > > > > It'll be good idea to fix order of other entries too. > > > > Happy to! Would you like that in this diff or a new one? > > > This is trivial change and could be made in same commit. > > Done > Sorry for being unclear, but I meant alphabetical order for check names, not > beginning of sentences. This is then unclear to me. How do you alphabetize by check name? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128402/new/ https://reviews.llvm.org/D128402 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
