[PATCH] D126495: [clang-tidy] Organize test docs into subdirectories by module (NFC)

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 9 10:34:49 PDT 2022


aaron.ballman added a comment.

In D126495#3569863 <https://reviews.llvm.org/D126495#3569863>, @LegalizeAdulthood wrote:

> In D126495#3569029 <https://reviews.llvm.org/D126495#3569029>, @njames93 wrote:
>
>> It would also be nice if there was a redirect that would dynamically translate the old links to new links.
>
> You can do that with `.htaccess`, but I don't know if that's considered acceptable in clang documentation.

FWIW, I don't have any idea if this is or isn't acceptable.

> Leaving `.rst` pages in place for all the old locations just recreates the clutter instead of getting rid of it.

+1

I don't feel super comfortable reviewing the Python changes, but I spot checked the changes and they all seem reasonable to me. So this has my LG, but I'm not accepting the review because of the Python bit.


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

https://reviews.llvm.org/D126495



More information about the cfe-commits mailing list