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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to