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