You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bookkeeper.apache.org by eo...@apache.org on 2017/05/18 14:21:50 UTC
bookkeeper git commit: BOOKKEEPER-1070: bk-merge-pr.py use
apache-rat:check goal
Repository: bookkeeper
Updated Branches:
refs/heads/master 0f16da896 -> 7da9ed293
BOOKKEEPER-1070: bk-merge-pr.py use apache-rat:check goal
Author: eolivelli <eo...@apache.org>
Reviewers: Jia Zhai <None>
Closes #158 from eolivelli/BOOKKEEPER-1070-rat
Project: http://git-wip-us.apache.org/repos/asf/bookkeeper/repo
Commit: http://git-wip-us.apache.org/repos/asf/bookkeeper/commit/7da9ed29
Tree: http://git-wip-us.apache.org/repos/asf/bookkeeper/tree/7da9ed29
Diff: http://git-wip-us.apache.org/repos/asf/bookkeeper/diff/7da9ed29
Branch: refs/heads/master
Commit: 7da9ed293db87103efcc28b7e706c0b386131ec1
Parents: 0f16da8
Author: Enrico Olivelli <eo...@apache.org>
Authored: Thu May 18 16:21:37 2017 +0200
Committer: eolivelli <eo...@apache.org>
Committed: Thu May 18 16:21:37 2017 +0200
----------------------------------------------------------------------
dev/bk-merge-pr.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/bookkeeper/blob/7da9ed29/dev/bk-merge-pr.py
----------------------------------------------------------------------
diff --git a/dev/bk-merge-pr.py b/dev/bk-merge-pr.py
index b59bfc5..cb5bd43 100755
--- a/dev/bk-merge-pr.py
+++ b/dev/bk-merge-pr.py
@@ -155,7 +155,7 @@ def merge_pr(pr_num, target_ref, title, body, default_pr_reviewers, pr_repo_desc
# Offer to run findbugs and rat before committing
result = raw_input('Do you want to validate findbugs and rat after the merge? (y/n): ')
if result.lower() == 'y':
- test_res = subprocess.call('mvn clean install -DskipTests findbugs:check rat:rat'.split())
+ test_res = subprocess.call('mvn clean install -DskipTests findbugs:check apache-rat:check'.split())
if test_res == 0:
print('QA tests execution succeeded')
else: