From commits-return-1399-archive-asf-public=cust-asf.ponee.io@yetus.apache.org Tue Dec 10 18:03:38 2019 Return-Path: X-Original-To: archive-asf-public@cust-asf.ponee.io Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [207.244.88.153]) by mx-eu-01.ponee.io (Postfix) with SMTP id B41F6180630 for ; Tue, 10 Dec 2019 19:03:37 +0100 (CET) Received: (qmail 85218 invoked by uid 500); 10 Dec 2019 18:03:37 -0000 Mailing-List: contact commits-help@yetus.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@yetus.apache.org Delivered-To: mailing list commits@yetus.apache.org Received: (qmail 85209 invoked by uid 99); 10 Dec 2019 18:03:36 -0000 Received: from ec2-52-202-80-70.compute-1.amazonaws.com (HELO gitbox.apache.org) (52.202.80.70) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 10 Dec 2019 18:03:36 +0000 Received: by gitbox.apache.org (ASF Mail Server at gitbox.apache.org, from userid 33) id 745448D809; Tue, 10 Dec 2019 18:03:36 +0000 (UTC) Date: Tue, 10 Dec 2019 18:03:36 +0000 To: "commits@yetus.apache.org" Subject: [yetus] branch master updated: YETUS-926. Add visual separation to console report summary (#81) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Message-ID: <157600101624.23998.3304698924129485635@gitbox.apache.org> From: ndimiduk@apache.org X-Git-Host: gitbox.apache.org X-Git-Repo: yetus X-Git-Refname: refs/heads/master X-Git-Reftype: branch X-Git-Oldrev: cb3ad2546cf289942db1398fa4e33e75113ce986 X-Git-Newrev: 118845e947ae22d4077dde50a911dce719eea00c X-Git-Rev: 118845e947ae22d4077dde50a911dce719eea00c X-Git-NotificationType: ref_changed_plus_diff X-Git-Multimail-Version: 1.5.dev Auto-Submitted: auto-generated This is an automated email from the ASF dual-hosted git repository. ndimiduk 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 118845e YETUS-926. Add visual separation to console report summary (#81) 118845e is described below commit 118845e947ae22d4077dde50a911dce719eea00c Author: Nick Dimiduk AuthorDate: Tue Dec 10 10:03:27 2019 -0800 YETUS-926. Add visual separation to console report summary (#81) The final output table is a wall of text. If you squint a little, you can see that there are subsections contained within. Add some visual separation between sections so that one can navigate the results. Before this change: ``` | Vote | Subsystem | Runtime | Comment ============================================================================ | | | | Prechecks | +1 | dupname | 0m 0s | No case conflicting files found. | +1 | hbaseanti | 0m 0s | Patch does not have any anti-patterns. | +1 | @author | 0m 0s | The patch does not contain any @author | | | | tags. | +1 | test4tests | 0m 0s | The patch appears to include 4 new or | | | | modified test files. | | | | master Compile Tests | +1 | mvninstall | 4m 6s | master passed | +1 | compile | 0m 31s | master passed | +1 | checkstyle | 0m 37s | master passed | +1 | shadedjars | 2m 30s | branch has no errors when building our | | | | shaded downstream artifacts. | +1 | javadoc | 0m 22s | master passed | 0 | spotbugs | 3m 20s | Used deprecated FindBugs config; | | | | considering switching to SpotBugs. | +1 | findbugs | 3m 19s | master passed | | | | Patch Compile Tests | +1 | mvninstall | 2m 49s | the patch passed | +1 | compile | 0m 30s | the patch passed | +1 | javac | 0m 30s | the patch passed | -0 | checkstyle | 0m 34s | hbase-server: The patch generated 8 new | | | | + 234 unchanged - 7 fixed = 242 total | | | | (was 241) | +1 | whitespace | 0m 0s | The patch has no whitespace issues. | +1 | shadedjars | 2m 31s | patch has no errors when building our | | | | shaded downstream artifacts. | +1 | hadoopcheck | 11m 22s | Patch does not cause any errors with | | | | Hadoop 2.8.5 2.9.2 or 3.1.1 3.1.2. | +1 | javadoc | 0m 18s | the patch passed | +1 | findbugs | 3m 19s | the patch passed | | | | Other Tests | -1 | unit | 26m 48s | hbase-server in the patch failed. | +1 | asflicense | 0m 22s | The patch does not generate ASF License | | | | warnings. | | | 62m 50s | ``` After: ``` | Vote | Subsystem | Runtime | Comment ============================================================================ +--------------------------------------------------------------------------- | | | | Prechecks +--------------------------------------------------------------------------- | +1 | dupname | 0m 0s | No case conflicting files found. | +1 | hbaseanti | 0m 0s | Patch does not have any anti-patterns. | +1 | @author | 0m 0s | The patch does not contain any @author | | | | tags. | +1 | test4tests | 0m 0s | The patch appears to include 4 new or | | | | modified test files. +--------------------------------------------------------------------------- | | | | master Compile Tests +--------------------------------------------------------------------------- | +1 | mvninstall | 3m 54s | master passed | +1 | compile | 0m 30s | master passed | +1 | checkstyle | 0m 37s | master passed | +1 | shadedjars | 2m 27s | branch has no errors when building our | | | | shaded downstream artifacts. | +1 | javadoc | 0m 23s | master passed | 0 | spotbugs | 3m 23s | Used deprecated FindBugs config; | | | | considering switching to SpotBugs. | +1 | findbugs | 3m 22s | master passed +--------------------------------------------------------------------------- | | | | Patch Compile Tests +--------------------------------------------------------------------------- | +1 | mvninstall | 2m 42s | the patch passed | +1 | compile | 0m 29s | the patch passed | +1 | javac | 0m 29s | the patch passed | -0 | checkstyle | 0m 34s | hbase-server: The patch generated 8 new | | | | + 234 unchanged - 7 fixed = 242 total | | | | (was 241) | +1 | whitespace | 0m 0s | The patch has no whitespace issues. | +1 | shadedjars | 2m 27s | patch has no errors when building our | | | | shaded downstream artifacts. | +1 | hadoopcheck | 11m 36s | Patch does not cause any errors with | | | | Hadoop 2.8.5 2.9.2 or 3.1.1 3.1.2. | +1 | javadoc | 0m 19s | the patch passed | +1 | findbugs | 3m 22s | the patch passed +--------------------------------------------------------------------------- | | | | Other Tests +--------------------------------------------------------------------------- | -1 | unit | 51m 52s | hbase-server in the patch failed. | +1 | asflicense | 0m 11s | The patch does not generate ASF License | | | | warnings. | | | 87m 43s | ``` Signed-off-by: Sean Busbey --- precommit/src/main/shell/core.d/builtin-bugsystem.sh | 2 ++ 1 file changed, 2 insertions(+) diff --git a/precommit/src/main/shell/core.d/builtin-bugsystem.sh b/precommit/src/main/shell/core.d/builtin-bugsystem.sh index 4294227..0caad61 100755 --- a/precommit/src/main/shell/core.d/builtin-bugsystem.sh +++ b/precommit/src/main/shell/core.d/builtin-bugsystem.sh @@ -124,7 +124,9 @@ function console_finalreport ${SED} -e '1d' "${commentfile1}" > "${commentfile2}" if [[ "${vote}" = "H" ]]; then + echo "+---------------------------------------------------------------------------" printf '| | %*s | |%-s\n' ${seccoladj} " " "${normaltop}" + echo "+---------------------------------------------------------------------------" else printf '| %4s | %*s | %-10s |%-s\n' "${vote}" ${seccoladj} \ "${subs}" "${calctime}" "${normaltop}"