This is an archive of the discontinued LLVM Phabricator instance.

[clang-tidy] Mention "add the check to release note" in add_new_check.py.
AbandonedPublic

Authored by hokein on Apr 7 2017, 3:20 AM.

Event Timeline

hokein created this revision.Apr 7 2017, 3:20 AM

I don't know how hard it'll be to implement, but will be great to add

  • New <check name> <http://clang.llvm.org/extra/clang-tidy/checks/<check name>.html>_ check

    Please add short description here.

into "Improvements to clang-tidy" section. Ideally according to alphabetic order.

Eugene.Zelenko set the repository for this revision to rL LLVM.
alexfh edited edge metadata.Jul 12 2017, 6:15 AM

I don't know how hard it'll be to implement, but will be great to add

  • New <check name> <http://clang.llvm.org/extra/clang-tidy/checks/<check name>.html>_ check

    Please add short description here.

Good idea. Implemented in r307787. No alphabetic order though. Feel free to fix ;)

into "Improvements to clang-tidy" section. Ideally according to alphabetic order.

alexfh resigned from this revision.Jul 12 2017, 6:15 AM
hokein abandoned this revision.Jul 12 2017, 7:17 AM

Good idea. Implemented in r307787. No alphabetic order though. Feel free to fix ;)

Looks good. Thanks!

Closing this revision.