diff options
author | Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com> | 2015-02-12 16:47:32 +0100 |
---|---|---|
committer | Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com> | 2015-02-13 13:59:19 +0000 |
commit | cbddd3a5a6c9022b456c0df0ae8cf30eddb8cde6 (patch) | |
tree | 5c878d9ed7c904135175b5111de72f1f43ed65cd /git-hooks/git_post_commit_hook | |
parent | 62a2168e83504eefec02598e995c87a1e83f3506 (diff) |
move the work-in-progress detection to the bot itself
it obviously makes no sense when the sanitizer is executed locally.
additionally, this allows us a different treatment of WIP reports:
instead of giving a -2 sanity review, give a -2 code review. that way
the change owner is not spammed by useless notifications (because the
sanity review itself is positive, so the message is suppressed by the
gerrit configuration).
Change-Id: I10bc8f8f61673e75983dcb91310836d59c434483
Reviewed-by: Orgad Shaneh <orgads@gmail.com>
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com>
Diffstat (limited to 'git-hooks/git_post_commit_hook')
-rwxr-xr-x | git-hooks/git_post_commit_hook | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/git-hooks/git_post_commit_hook b/git-hooks/git_post_commit_hook index f4bd82a..6550d92 100755 --- a/git-hooks/git_post_commit_hook +++ b/git-hooks/git_post_commit_hook @@ -39,6 +39,4 @@ if test -L "$me"; then fi sha1=${1-HEAD} # just for debugging -GIT_PUSH=${GIT_PUSH+$GIT_PUSH,}wip # this check makes totally no sense locally -export GIT_PUSH exec `dirname "$me"`/sanitize-commit $sha1 strict >&2 |