[PATCH] D15796: [PATCH] clang-tidy documentation redirects
Alexander Kornienko via cfe-commits
cfe-commits at lists.llvm.org
Mon Jan 11 06:43:29 PST 2016
alexfh added a comment.
Sorry for the delay and thanks for pinging me.
Comment at: docs/clang-tidy/checks/cert-dcl54-cpp.rst:6
@@ +5,3 @@
> > We need to teach add_new_check.py to retain (or automatically generate) these comments then.
> I have not touched add_new_check.py for this because add_new_check.py knows nothing about check aliases anyway. Whenever we update add_new_check.py to generate checker aliases directly, that would be a good time to do this work.
add_new_check.py should do something reasonable with list.rst right away, otherwise the `(redirects to ...)` comments will be lost on each run of the script, which will result in annoying manual work of reverting these changes. Do you need help implementing alias handling in the add_new_check.py script?
Comment at: docs/clang-tidy/checks/cert-dcl54-cpp.rst:11
@@ +10,1 @@
\ No newline at end of file
nit: Could you add newlines at the end of files to remove this annoying line?
More information about the cfe-commits