[PATCH] D18509: clang-tidy: add_new_check.py stubs out release notes
Alexander Kornienko via cfe-commits
cfe-commits at lists.llvm.org
Thu Mar 31 05:25:48 PDT 2016
alexfh added a comment.
I like the idea, but we could make this a bit more reliable, see the inline comment.
================
Comment at: clang-tidy/add_new_check.py:280
@@ +279,3 @@
+ if not clang_tidy_found:
+ if line.startswith("Improvements to ``clang-tidy``"):
+ clang_tidy_found = True
----------------
Maybe instead add a special placeholder to the .rst file (something like `<describe new clang-tidy checks here>`) and insert new text right _before_ it? This will solve two problems: dependency on the formatting of the text and chronological order of the notes.
http://reviews.llvm.org/D18509
More information about the cfe-commits
mailing list