[
https://issues.apache.org/jira/browse/YETUS-995?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17198871#comment-17198871
]
Allen Wittenauer commented on YETUS-995:
----------------------------------------
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Logfile || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m
2s{color} | {color:blue}{color} | {color:blue} Precommit patch detected.
{color} |
|| || || || {color:brown} Prechecks {color} || ||
| {color:green}+1{color} | {color:green} dupname {color} | {color:green} 0m
1s{color} | {color:green}{color} | {color:green} No case conflicting files
found. {color} |
|| || || || {color:brown} Compile Tests {color} || ||
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue} 0m
8s{color} | {color:blue}{color} | {color:blue} Maven dependency ordering
{color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 8m
20s{color} | {color:green}{color} | {color:green} the source passed {color} |
| {color:green}+1{color} | {color:green} jshint {color} | {color:green} 0m
3s{color} | {color:green}{color} | {color:green} There were no new jshint
issues. {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m
45s{color} | {color:green}{color} | {color:green} the source passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green} 0m
45s{color} | {color:green}{color} | {color:green} the source passed {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m
14s{color} | {color:green}{color} | {color:green} The source tree does not
contain any @author tags. {color} |
| {color:orange}-0{color} | {color:orange} checkstyle {color} | {color:orange}
0m 9s{color} | {color:orange}
https://1147-151960188-gh.circle-artifacts.com/0/tmp/yetus-out/diff-checkstyle-root.txt{color}
| {color:orange} root: The source tree has 93 issues. {color} |
| {color:green}+1{color} | {color:green} hadolint {color} | {color:green} 0m
3s{color} | {color:green}{color} | {color:green} There were no new hadolint
issues. {color} |
| {color:green}+1{color} | {color:green} markdownlint {color} | {color:green}
0m 6s{color} | {color:green}{color} | {color:green} There were no new
markdownlint issues. {color} |
| {color:green}+1{color} | {color:green} pylint {color} | {color:green} 0m
12s{color} | {color:green}{color} | {color:green} There were no new pylint
issues. {color} |
| {color:green}+1{color} | {color:green} rubocop {color} | {color:green} 0m
7s{color} | {color:green}{color} | {color:green} There were no new rubocop
issues. {color} |
| {color:red}-1{color} | {color:red} shellcheck {color} | {color:red} 0m
27s{color} | {color:red}
https://1147-151960188-gh.circle-artifacts.com/0/tmp/yetus-out/diff-patch-shellcheck.txt{color}
| {color:red} The source tree has 4 issues. {color} |
| {color:green}+1{color} | {color:green} shelldocs {color} | {color:green} 0m
4s{color} | {color:green}{color} | {color:green} There were no new shelldocs
issues. {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m
0s{color} | {color:green}{color} | {color:green} The source tree has no
whitespace issues. {color} |
| {color:green}+1{color} | {color:green} xml {color} | {color:green} 0m
26s{color} | {color:green}{color} | {color:green} The source tree has no
ill-formed XML file. {color} |
| {color:green}+1{color} | {color:green} yamllint {color} | {color:green} 0m
2s{color} | {color:green}{color} | {color:green} There were no new yamllint
issues. {color} |
| {color:orange}-0{color} | {color:orange} javadoc {color} | {color:orange} 0m
16s{color} | {color:orange}
https://1147-151960188-gh.circle-artifacts.com/0/tmp/yetus-out/diff-javadoc-javadoc-root.txt{color}
| {color:orange} root has 1 issues. {color} |
| {color:blue}0{color} | {color:blue} spotbugs {color} | {color:blue} 1m
47s{color} | {color:blue}{color} | {color:blue} Both FindBugs and SpotBugs are
enabled, using SpotBugs. {color} |
| {color:green}+1{color} | {color:green} spotbugs {color} | {color:green} 1m
30s{color} | {color:green}{color} | {color:green} the source passed {color} |
|| || || || {color:brown} Other Tests {color} || ||
| {color:green}+1{color} | {color:green} unit {color} | {color:green} 1m
37s{color} | {color:green}{color} | {color:green} root in the source passed.
{color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m
7s{color} | {color:green}{color} | {color:green} The source tree does not
generate ASF License warnings. {color} |
| {color:black}{color} | {color:black} {color} | {color:black} 14m 47s{color} |
{color:black}{color} | {color:black}{color} |
\\
\\
|| Subsystem || Report/Notes ||
| Optional Tests | dupname asflicense yamllint jshint shellcheck shelldocs
hadolint rubocop markdownlint javac javadoc unit xml compile spotbugs findbugs
checkstyle pylint |
| uname | Linux ab90cba5644e 4.15.0-1077-aws #81-Ubuntu SMP Wed Jun 24 16:48:15
UTC 2020 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| git revision | yetus995 / 49855ea |
| Default Java | Private Build-1.8.0_265-8u265-b01-0ubuntu2~20.04-b01 |
| modules | C: audience-annotations-component/audience-annotations
yetus-assemblies audience-annotations-component yetus-dist . asf-site-src
audience-annotations-component/audience-annotations-jdiff precommit
releasedocmaker shelldocs yetus-maven-plugin yetus-minimaven-plugin U: . |
| Console output | https://circleci.com/gh/aw-was-here/buretoolbox/1147 |
| versions | git=2.25.1 maven=3.6.3 jshint=2.12.0 hadolint=1.18.0-0-g76eee5c
markdownlint=0.23.2 pylint=1.9.2 rubocop=0.90.0 yamllint=1.24.2 spotbugs=4.0.4 |
| Powered by | Apache Yetus 0.13.0-SNAPSHOT https://yetus.apache.org |
This message was automatically generated.
> vote table should have access to the log filename
> -------------------------------------------------
>
> Key: YETUS-995
> URL: https://issues.apache.org/jira/browse/YETUS-995
> Project: Yetus
> Issue Type: Improvement
> Reporter: Allen Wittenauer
> Assignee: Allen Wittenauer
> Priority: Major
> Fix For: 0.13.0
>
>
> A big design from early on in test-patch history was that the log entry was
> separate from the vote. This setup was likely done for a variety of
> aesthetic and practical reasons for JIRA. However, that design is flawed in
> that it is letting display dictate internals. With GitHub Status
> (YETUS-994), this flaw is now very very obvious.
> Fixing this flaw is going to be a MAJOR compatibility break with no
> particularly easy way to make existing code work. Details below.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)