[PATCH] D126495: [clang-tidy] Organize test docs into subdirectories by module (NFC)
Richard via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jun 16 10:48:15 PDT 2022
LegalizeAdulthood added a comment.
In D126495#3588787 <https://reviews.llvm.org/D126495#3588787>, @aaron.ballman wrote:
> In D126495#3574115 <https://reviews.llvm.org/D126495#3574115>, @LegalizeAdulthood wrote:
>
>> In D126495#3571511 <https://reviews.llvm.org/D126495#3571511>, @njames93 wrote:
>>
>>> Search engines isn't really an issue, its more people who are using older versions of llvm that try and go to the documentation which turns out is a dead link.
>>> Having said that, I still think the lack of a redirect shouldn't block this change.
>>
>> Aren't we archiving old versions of the documentation, though?
>
> We do:
>
> https://releases.llvm.org/12.0.0/tools/clang/tools/extra/docs/
> https://releases.llvm.org/11.0.0/tools/clang/tools/extra/docs/
> and so on.
>
> Giving my LG more officially now to unblock this, but I'd still appreciate some extra eyes on the Python changes.
I tested by generating a new check and validating the links in the documentation as well as running `--update-docs` and checking that the updates were correct. So functionally, there should be no problem and I tried to follow the existing style of the file, so should be good on style nits as well.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D126495/new/
https://reviews.llvm.org/D126495
More information about the cfe-commits
mailing list