[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