Todd Lipcon has submitted this change and it was merged.

Change subject: Add script to run clang-tidy against a patch
......................................................................


Add script to run clang-tidy against a patch

This tooling runs clang-tidy against a given patch, and formats the
resulting warnings in a form that can be consumed by the gerrit API[1]

This has been running out of a private branch for many months and
proven generally useful. This patch also updates .clang-tidy based on
the configuration that has been running.

[1] 
https://gerrit-review.googlesource.com/Documentation/rest-api-changes.html#set-review

Change-Id: I6c6246f4691b0ca143eea4aa8fb6a4b23099ce4e
Reviewed-on: http://gerrit.cloudera.org:8080/4381
Reviewed-by: Adar Dembo <a...@cloudera.com>
Tested-by: Kudu Jenkins
---
M .ycm_extra_conf.py
A build-support/clang_tidy_gerrit.py
A build-support/compile_flags.py
M src/kudu/.clang-tidy
4 files changed, 241 insertions(+), 33 deletions(-)

Approvals:
  Adar Dembo: Looks good to me, approved
  Kudu Jenkins: Verified



-- 
To view, visit http://gerrit.cloudera.org:8080/4381
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I6c6246f4691b0ca143eea4aa8fb6a4b23099ce4e
Gerrit-PatchSet: 5
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mpe...@apache.org>
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>

Reply via email to