This is an automated email from the ASF dual-hosted git repository.

aw pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/yetus.git


The following commit(s) were added to refs/heads/master by this push:
     new cbbd49a  YETUS-903. revive/golang/golanci aren't removing options from 
the stack (#71)
cbbd49a is described below

commit cbbd49a957a685c94cc431916de1c697e49f66cd
Author: Allen Wittenauer <a...@apache.org>
AuthorDate: Wed Aug 14 21:43:12 2019 -0700

    YETUS-903. revive/golang/golanci aren't removing options from the stack 
(#71)
---
 precommit/src/main/shell/test-patch.d/golang.sh   | 1 +
 precommit/src/main/shell/test-patch.d/golangci.sh | 2 ++
 precommit/src/main/shell/test-patch.d/revive.sh   | 2 ++
 3 files changed, 5 insertions(+)

diff --git a/precommit/src/main/shell/test-patch.d/golang.sh 
b/precommit/src/main/shell/test-patch.d/golang.sh
index 2b67365..ec6717e 100755
--- a/precommit/src/main/shell/test-patch.d/golang.sh
+++ b/precommit/src/main/shell/test-patch.d/golang.sh
@@ -42,6 +42,7 @@ function golang_parse_args
     case ${i} in
       --golang-go=*)
         GOEXE=${i#*=}
+        delete_parameter "${i}"
       ;;
     esac
   done
diff --git a/precommit/src/main/shell/test-patch.d/golangci.sh 
b/precommit/src/main/shell/test-patch.d/golangci.sh
index 462dbb0..671cab4 100644
--- a/precommit/src/main/shell/test-patch.d/golangci.sh
+++ b/precommit/src/main/shell/test-patch.d/golangci.sh
@@ -43,9 +43,11 @@ function golangcilint_parse_args
     case ${i} in
       --golangcilint=*)
         GOLANGCI_LINT=${i#*=}
+        delete_parameter "${i}"
       ;;
       --golangcilint-config=*)
         GOLANGCI_CONFIG=${i#*=}
+        delete_parameter "${i}"
       ;;
     esac
   done
diff --git a/precommit/src/main/shell/test-patch.d/revive.sh 
b/precommit/src/main/shell/test-patch.d/revive.sh
index 4c5ef95..71901e4 100755
--- a/precommit/src/main/shell/test-patch.d/revive.sh
+++ b/precommit/src/main/shell/test-patch.d/revive.sh
@@ -36,9 +36,11 @@ function revive_parse_args
     case ${i} in
       --revive=*)
         REVIVE=${i#*=}
+        delete_parameter "${i}"
       ;;
       --revive-config=*)
         REVIVE_CONFIG=${i#*=}
+        delete_parameter "${i}"
       ;;
     esac
   done

Reply via email to