[kudu-CR] Add "make tidy" target

2017-08-31 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: Add "make tidy" target .. Add "make tidy" target This adds a simple cmake target to run a local clang-tidy check, instead of having to come up with the inc

[kudu-CR] Add "make tidy" target

2017-08-31 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Add "make tidy" target .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7917 To unsubscribe, visit http://gerrit.cloudera.org:808

[kudu-CR] Add "make tidy" target

2017-08-31 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Add "make tidy" target .. Patch Set 2: Code-Review+1 (1 comment) http://gerrit.cloudera.org:8080/#/c/7917/2/README.adoc File README.adoc: Line 263: === Running

[kudu-CR] Add "make tidy" target

2017-08-31 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add "make tidy" target .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/7917/2/CMakeLists.txt File CMakeLists.txt: Line 1117: add_custom_target(

[kudu-CR] Add "make tidy" target

2017-08-31 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Add "make tidy" target .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/7917/2/CMakeLists.txt File CMakeLists.txt: Line 1118: add_dependencie

[kudu-CR] Add "make tidy" target

2017-08-31 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Add "make tidy" target .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/7917/2/CMakeLists.txt File CMakeLists.txt: Line 1117: add_custom_target(

[kudu-CR] Add "make tidy" target

2017-08-31 Thread Mike Percy (Code Review)
Hello Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7917 to look at the new patch set (#2). Change subject: Add "make tidy" target .. Add "make tidy" target

[kudu-CR] Add "make tidy" target

2017-08-31 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Add "make tidy" target .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7917 To unsubscribe, visit http://gerrit.cloudera.org:808

[kudu-CR] Add "make tidy" target

2017-08-31 Thread Mike Percy (Code Review)
Mike Percy has uploaded a new change for review. http://gerrit.cloudera.org:8080/7917 Change subject: Add "make tidy" target .. Add "make tidy" target This adds a simple cmake target to run a local clang-tidy check, instead o