[kudu-CR] Add a basic .clang-tidy configuration

2016-09-19 Thread Todd Lipcon (Code Review)
Hello David Ribeiro Alves, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4453 to review the following change. Change subject: Add a basic .clang-tidy configuration .. Add a basic .clang-ti

[kudu-CR] Add a basic .clang-tidy configuration

2016-09-19 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Add a basic .clang-tidy configuration .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4453/1/src/kudu/.clang-tidy File src/kudu/.clang-tidy: Line

[kudu-CR] Add a basic .clang-tidy configuration

2016-09-19 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add a basic .clang-tidy configuration .. Patch Set 1: any docs on how to use this locally? -- To view, visit http://gerrit.cloudera.org:8080/4453 To unsu

[kudu-CR] Add a basic .clang-tidy configuration

2016-09-19 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add a basic .clang-tidy configuration .. Patch Set 1: (1 comment) Will add some scripting in a future commit for running this locally. http://gerrit.cloudera.org

[kudu-CR] Add a basic .clang-tidy configuration

2016-09-23 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add a basic .clang-tidy configuration .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4453 To unsubscribe, visit http://ge

[kudu-CR] Add a basic .clang-tidy configuration

2016-09-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Add a basic .clang-tidy configuration .. Add a basic .clang-tidy configuration This configuration enables just the most important checks for now, and disa