[kudu-CR] Add script to run clang-tidy against a patch

2017-06-05 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add script to run clang-tidy against a patch .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/4381/3/build-support/compile_flags.py File

[kudu-CR] Add script to run clang-tidy against a patch

2017-06-05 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Add script to run clang-tidy against a patch .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4381 To unsubscribe, visit

[kudu-CR] Add script to run clang-tidy against a patch

2017-06-05 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Add script to run clang-tidy against a patch .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/4381/3/build-support/compile_flags.py File

[kudu-CR] Add script to run clang-tidy against a patch

2017-06-05 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4381 to look at the new patch set (#4). Change subject: Add script to run clang-tidy against a patch .. Add script to

[kudu-CR] Add script to run clang-tidy against a patch

2017-06-05 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add script to run clang-tidy against a patch .. Patch Set 3: (6 comments) > Forgot to ask: how is clang_tidy_gerrit.py invoked? There's a jenkins job here:

[kudu-CR] Add script to run clang-tidy against a patch

2016-10-05 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Add script to run clang-tidy against a patch .. Patch Set 3: Forgot to ask: how is clang_tidy_gerrit.py invoked? -- To view, visit

[kudu-CR] Add script to run clang-tidy against a patch

2016-10-05 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Add script to run clang-tidy against a patch .. Patch Set 3: (6 comments) http://gerrit.cloudera.org:8080/#/c/4381/3/build-support/clang_tidy_gerrit.py File

[kudu-CR] Add script to run clang-tidy against a patch

2016-10-04 Thread Todd Lipcon (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4381 to look at the new patch set (#3). Change subject: Add script to run clang-tidy against a patch .. Add