You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@yetus.apache.org by aw...@apache.org on 2016/04/20 01:10:46 UTC
[1/2] yetus git commit: YETUS-384. In build driver mode,
some plugins have misleading status text
Repository: yetus
Updated Branches:
refs/heads/YETUS-156 f4159bd06 -> 3f782d346
YETUS-384. In build driver mode, some plugins have misleading status text
Signed-off-by: Allen Wittenauer <aw...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/yetus/repo
Commit: http://git-wip-us.apache.org/repos/asf/yetus/commit/088070a7
Tree: http://git-wip-us.apache.org/repos/asf/yetus/tree/088070a7
Diff: http://git-wip-us.apache.org/repos/asf/yetus/diff/088070a7
Branch: refs/heads/YETUS-156
Commit: 088070a7b1283ef1d8c1e57ba2e887e2bf2bb3ae
Parents: f4159bd
Author: Allen Wittenauer <aw...@apache.org>
Authored: Mon Apr 18 13:44:58 2016 -0700
Committer: Allen Wittenauer <aw...@apache.org>
Committed: Tue Apr 19 15:54:13 2016 -0700
----------------------------------------------------------------------
precommit/test-patch.d/perlcritic.sh | 4 ++--
precommit/test-patch.d/pylint.sh | 4 ++--
precommit/test-patch.d/rubocop.sh | 4 ++--
precommit/test-patch.d/ruby-lint.sh | 4 ++--
4 files changed, 8 insertions(+), 8 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/yetus/blob/088070a7/precommit/test-patch.d/perlcritic.sh
----------------------------------------------------------------------
diff --git a/precommit/test-patch.d/perlcritic.sh b/precommit/test-patch.d/perlcritic.sh
index b376347..291e94e 100755
--- a/precommit/test-patch.d/perlcritic.sh
+++ b/precommit/test-patch.d/perlcritic.sh
@@ -68,7 +68,7 @@ function perlcritic_preapply
start_clock
- echo "Running perlcritic against modified perl scripts/modules."
+ echo "Running perlcritic against identified perl scripts/modules."
pushd "${BASEDIR}" >/dev/null
for i in "${CHANGED_FILES[@]}"; do
if [[ ${i} =~ \.p[lm]$ && -f ${i} ]]; then
@@ -114,7 +114,7 @@ function perlcritic_postapply
# by setting the clock back
offset_clock "${PERLCRITIC_TIMER}"
- echo "Running perlcritic against modified perl scripts/modules."
+ echo "Running perlcritic against identified perl scripts/modules."
# we re-check this in case one has been added
pushd "${BASEDIR}" >/dev/null
for i in "${CHANGED_FILES[@]}"; do
http://git-wip-us.apache.org/repos/asf/yetus/blob/088070a7/precommit/test-patch.d/pylint.sh
----------------------------------------------------------------------
diff --git a/precommit/test-patch.d/pylint.sh b/precommit/test-patch.d/pylint.sh
index 0f1790f..7decb96 100755
--- a/precommit/test-patch.d/pylint.sh
+++ b/precommit/test-patch.d/pylint.sh
@@ -75,7 +75,7 @@ function pylint_preapply
start_clock
- echo "Running pylint against modified python scripts."
+ echo "Running pylint against identified python scripts."
pushd "${BASEDIR}" >/dev/null
for i in "${CHANGED_FILES[@]}"; do
if [[ ${i} =~ \.py$ && -f ${i} ]]; then
@@ -121,7 +121,7 @@ function pylint_postapply
# by setting the clock back
offset_clock "${PYLINT_TIMER}"
- echo "Running pylint against modified python scripts."
+ echo "Running pylint against identified python scripts."
# we re-check this in case one has been added
pushd "${BASEDIR}" >/dev/null
for i in "${CHANGED_FILES[@]}"; do
http://git-wip-us.apache.org/repos/asf/yetus/blob/088070a7/precommit/test-patch.d/rubocop.sh
----------------------------------------------------------------------
diff --git a/precommit/test-patch.d/rubocop.sh b/precommit/test-patch.d/rubocop.sh
index f73c073..828bea1 100755
--- a/precommit/test-patch.d/rubocop.sh
+++ b/precommit/test-patch.d/rubocop.sh
@@ -68,7 +68,7 @@ function rubocop_preapply
start_clock
- echo "Running rubocop against modified ruby scripts."
+ echo "Running rubocop against identified ruby scripts."
pushd "${BASEDIR}" >/dev/null
for i in "${CHANGED_FILES[@]}"; do
if [[ ${i} =~ \.rb$ && -f ${i} ]]; then
@@ -114,7 +114,7 @@ function rubocop_postapply
# by setting the clock back
offset_clock "${RUBOCOP_TIMER}"
- echo "Running rubocop against modified ruby scripts."
+ echo "Running rubocop against identified ruby scripts."
# we re-check this in case one has been added
pushd "${BASEDIR}" >/dev/null
for i in "${CHANGED_FILES[@]}"; do
http://git-wip-us.apache.org/repos/asf/yetus/blob/088070a7/precommit/test-patch.d/ruby-lint.sh
----------------------------------------------------------------------
diff --git a/precommit/test-patch.d/ruby-lint.sh b/precommit/test-patch.d/ruby-lint.sh
index c57887b..78be7b6 100755
--- a/precommit/test-patch.d/ruby-lint.sh
+++ b/precommit/test-patch.d/ruby-lint.sh
@@ -67,7 +67,7 @@ function ruby_lint_preapply
start_clock
- echo "Running ruby-lint against modified ruby scripts."
+ echo "Running ruby-lint against identified ruby scripts."
pushd "${BASEDIR}" >/dev/null
for i in "${CHANGED_FILES[@]}"; do
if [[ ${i} =~ \.rb$ && -f ${i} ]]; then
@@ -144,7 +144,7 @@ function ruby_lint_postapply
# by setting the clock back
offset_clock "${RUBY_LINT_TIMER}"
- echo "Running ruby-lint against modified ruby scripts."
+ echo "Running ruby-lint against identified ruby scripts."
# we re-check this in case one has been added
pushd "${BASEDIR}" >/dev/null
for i in "${CHANGED_FILES[@]}"; do
[2/2] yetus git commit: YETUS-385. test plugin for author tag
detection should have a way to opt-out of specific instances
Posted by aw...@apache.org.
YETUS-385. test plugin for author tag detection should have a way to opt-out of specific instances
Signed-off-by: Allen Wittenauer <aw...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/yetus/repo
Commit: http://git-wip-us.apache.org/repos/asf/yetus/commit/3f782d34
Tree: http://git-wip-us.apache.org/repos/asf/yetus/tree/3f782d34
Diff: http://git-wip-us.apache.org/repos/asf/yetus/diff/3f782d34
Branch: refs/heads/YETUS-156
Commit: 3f782d3460fbfa9f01de96b22e1e843b8851d240
Parents: 088070a
Author: Allen Wittenauer <aw...@apache.org>
Authored: Mon Apr 18 13:40:37 2016 -0700
Committer: Allen Wittenauer <aw...@apache.org>
Committed: Tue Apr 19 16:10:00 2016 -0700
----------------------------------------------------------------------
precommit/test-patch.d/author.sh | 43 ++++++++++++++++++++++++++++++++---
1 file changed, 40 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/yetus/blob/3f782d34/precommit/test-patch.d/author.sh
----------------------------------------------------------------------
diff --git a/precommit/test-patch.d/author.sh b/precommit/test-patch.d/author.sh
index 0e00c80..dccd504 100755
--- a/precommit/test-patch.d/author.sh
+++ b/precommit/test-patch.d/author.sh
@@ -16,6 +16,32 @@
add_test_type author
+## @description author usage hook
+## @audience private
+## @stability evolving
+## @replaceable no
+function author_usage
+{
+ yetus_add_option "--author-ignore-list=<list>" "list of filenames to ignore (full build mode only)"
+}
+
+## @description author parse args hook
+## @audience private
+## @stability evolving
+## @replaceable no
+function author_parse_args
+{
+ declare i
+
+ for i in "$@"; do
+ case ${i} in
+ --author-ignore-list=*)
+ yetus_comma_to_array AUTHOR_IGNORE_LIST "${i#*=}"
+ ;;
+ esac
+ done
+}
+
## @description helper function for @author tags check
## @audience private
## @stability evolving
@@ -93,7 +119,7 @@ function author_postcompile
{
# shellcheck disable=SC2155
declare -r appname=$(basename "${BASH_SOURCE-$0}")
- declare i
+ declare fn
if [[ "${BUILDMODE}" != full ]]; then
return
@@ -103,9 +129,20 @@ function author_postcompile
start_clock
- "${GIT}" grep -n -I --extended-regexp -i '^[^-].*@author' \
+ "${GIT}" grep -n -I --extended-regexp -i -e '^[^-].*@author' \
| ${GREP} -v "${appname}" \
- >> "${PATCH_DIR}/author-tags.txt"
+ >> "${PATCH_DIR}/author-tags-git.txt"
+
+ if [[ -z "${AUTHOR_IGNORE_LIST[0]}" ]]; then
+ cp -p "${PATCH_DIR}/author-tags-git.txt" "${PATCH_DIR}/author-tags.txt"
+ else
+ cp -p "${PATCH_DIR}/author-tags-git.txt" "${PATCH_DIR}/author-tags.1"
+ for fn in "${AUTHOR_IGNORE_LIST[@]}"; do
+ ${GREP} -v -E "^${fn}" "${PATCH_DIR}/author-tags.1" >> "${PATCH_DIR}/author-tags.2"
+ mv "${PATCH_DIR}/author-tags.2" "${PATCH_DIR}/author-tags.1"
+ done
+ mv "${PATCH_DIR}/author-tags.1" "${PATCH_DIR}/author-tags.txt"
+ fi
author_generic
}