Alexey Serbin has posted comments on this change. Change subject: [tidy] updated the 'tidy' target ......................................................................
Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7947/2/build-support/clang_tidy_gerrit.py File build-support/clang_tidy_gerrit.py: Line 169: print >>sys.stderr, "--rev-range works only with --no-gerrit" > Because I wanted that to work also for non-committed changes. Specifying ' oh, and it seems the description for the '--rev-range' is outdated -- I didn't reflect the fact it picks up local changes. -- To view, visit http://gerrit.cloudera.org:8080/7947 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f8db9076d5113ac44fb8200d00965f120126ac4 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: Yes
