Re: [PATCH] D13352: [PATCH] Add a CERT category for clang-tidy checkers

2015-10-02 Thread Aaron Ballman via cfe-commits
On Fri, Oct 2, 2015 at 9:59 AM, Manuel Klimek wrote: > I dont think we need finer grained code owners, but I also don't have real > objections. Then we can leave well enough alone. :-) ~Aaron > > > On Fri, Oct 2, 2015, 3:31 PM Aaron Ballman wrote: >> >> aaron.ballman closed this revision. >> a

Re: [PATCH] D13352: [PATCH] Add a CERT category for clang-tidy checkers

2015-10-02 Thread Manuel Klimek via cfe-commits
I dont think we need finer grained code owners, but I also don't have real objections. On Fri, Oct 2, 2015, 3:31 PM Aaron Ballman wrote: > aaron.ballman closed this revision. > aaron.ballman added a comment. > > Thanks! I've commit in r249130. > > Do we want to have code owners for this sort of

Re: [PATCH] D13352: [PATCH] Add a CERT category for clang-tidy checkers

2015-10-02 Thread Aaron Ballman via cfe-commits
aaron.ballman closed this revision. aaron.ballman added a comment. Thanks! I've commit in r249130. Do we want to have code owners for this sort of thing, or is that too fine-grained of a concept? If we do want them, I am (obviously) happy to be the code owner for anything in the CERT directory.

Re: [PATCH] D13352: [PATCH] Add a CERT category for clang-tidy checkers

2015-10-02 Thread Manuel Klimek via cfe-commits
klimek added a subscriber: klimek. klimek accepted this revision. klimek added a reviewer: klimek. klimek added a comment. This revision is now accepted and ready to land. lg http://reviews.llvm.org/D13352 ___ cfe-commits mailing list cfe-commits@li