[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