[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