You are viewing a plain text version of this content. The canonical link for it is here.
Posted to gitbox@yetus.apache.org by aw...@apache.org on 2020/10/29 15:54:27 UTC

[yetus] branch main updated: YETUS-1050. perlcritic should not force --brutal (#176)

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

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


The following commit(s) were added to refs/heads/main by this push:
     new 7e5ae70  YETUS-1050. perlcritic should not force --brutal (#176)
7e5ae70 is described below

commit 7e5ae7037c2cc522cb6aa44be838fa4ae242f2e8
Author: Allen Wittenauer <aw...@apache.org>
AuthorDate: Thu Oct 29 08:54:20 2020 -0700

    YETUS-1050. perlcritic should not force --brutal (#176)
---
 precommit/src/main/shell/test-patch.d/perlcritic.sh | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/precommit/src/main/shell/test-patch.d/perlcritic.sh b/precommit/src/main/shell/test-patch.d/perlcritic.sh
index fa380c8..7085329 100755
--- a/precommit/src/main/shell/test-patch.d/perlcritic.sh
+++ b/precommit/src/main/shell/test-patch.d/perlcritic.sh
@@ -75,7 +75,7 @@ function perlcritic_preapply
   pushd "${BASEDIR}" >/dev/null || return 1
   for i in "${CHANGED_FILES[@]}"; do
     if [[ ${i} =~ \.p[lm]$ && -f ${i} ]]; then
-      ${PERLCRITIC} -1 --verbose 1 "${i}" 2>/dev/null >> "${PATCH_DIR}/branch-perlcritic-result.txt"
+      "${PERLCRITIC}" --verbose 1 --nocolor "${i}" 2>/dev/null >> "${PATCH_DIR}/branch-perlcritic-result.txt"
     fi
   done
   popd >/dev/null || return 1
@@ -117,7 +117,7 @@ function perlcritic_postapply
   pushd "${BASEDIR}" >/dev/null || return 1
   for i in "${CHANGED_FILES[@]}"; do
     if [[ ${i} =~ \.p[lm]$ && -f ${i} ]]; then
-      ${PERLCRITIC} -1 --verbose 1 "${i}" 2>/dev/null >> "${PATCH_DIR}/patch-perlcritic-result.txt"
+      "${PERLCRITIC}" --verbose 1 --nocolor "${i}" 2>/dev/null >> "${PATCH_DIR}/patch-perlcritic-result.txt"
     fi
   done
   popd >/dev/null || return 1