[ 
https://issues.apache.org/jira/browse/YETUS-809?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16791729#comment-16791729
 ] 

Allen Wittenauer commented on YETUS-809:
----------------------------------------

{code}
| Vote |      Subsystem |  Runtime   | Comment
============================================================================
|   0  |        reexec  |   0m  5s   | Precommit patch detected. 
|      |                |            | Prechecks 
|  +1  |       dupname  |   0m  0s   | No case conflicting files found. 
|      |                |            | Compile Tests 
|   0  |        mvndep  |   0m  5s   | Maven dependency ordering 
|  +1  |    mvninstall  |   2m  5s   | the source passed 
|  +1  |        jshint  |   0m  2s   | There were no new jshint issues. 
|  +1  |       compile  |   0m 52s   | the source passed 
|  +1  |         javac  |   0m 52s   | the source passed 
|  +1  |       @author  |   0m 47s   | The source tree does not contain any 
|      |                |            | @author tags.
|  -0  |    checkstyle  |   0m  4s   | 
|      |                |            | 
audience-annotations-component/audience-an
|      |                |            | notations: The source tree has 87 issues.
|  -0  |    checkstyle  |   0m  3s   | 
|      |                |            | 
audience-annotations-component/audience-an
|      |                |            | notations-jdiff: The source tree has 11
|      |                |            | issues.
|  -0  |    checkstyle  |   0m  5s   | audience-annotations-component: The 
|      |                |            | source tree has 98 issues.
|  -0  |    checkstyle  |   0m 10s   | releasedocmaker: The source tree has 4 
|      |                |            | issues.
|  -0  |    checkstyle  |   0m  5s   | shelldocs: The source tree has 4 issues. 
|  -0  |    checkstyle  |   0m  6s   | yetus-maven-plugin: The source tree has 
|      |                |            | 10 issues.
|  -0  |    checkstyle  |   0m  7s   | root: The source tree has 116 issues. 
|  +1  |      hadolint  |   0m  2s   | There were no new hadolint issues. 
|  +1  |        pylint  |   0m  9s   | There were no new pylint issues. 
|  -0  |       rubocop  |   0m  3s   | The source tree has 13 issues. 
|  +1  |    shellcheck  |   0m  7s   | There were no new shellcheck issues. 
|  +1  |     shelldocs  |   0m  2s   | There were no new shelldocs issues. 
|  +1  |    whitespace  |   0m  0s   | The source tree has no whitespace 
|      |                |            | issues.
|  +1  |           xml  |   0m 27s   | The source tree has no ill-formed XML 
|      |                |            | file.
|  +1  |      yamllint  |   0m  1s   | There were no new yamllint issues. 
|   0  |      findbugs  |   0m  0s   | Skipped 6 modules in the source tree 
|      |                |            | with no Java source.
|  -1  |      findbugs  |   0m 11s   | 
|      |                |            | 
audience-annotations-component/audience-an
|      |                |            | notations in YETUS-809 has 1 extant
|      |                |            | Findbugs warnings.
|  -1  |      findbugs  |   0m 18s   | yetus-maven-plugin in YETUS-809 has 1 
|      |                |            | extant Findbugs warnings.
|  -0  |       javadoc  |   0m  4s   | yetus-maven-plugin has 1 issues. 
|  -0  |       javadoc  |   0m  7s   | root has 1 issues. 
|      |                |            | Other Tests 
|  +1  |          unit  |   0m  3s   | audience-annotations in the source 
|      |                |            | passed.
|  +1  |          unit  |   0m  8s   | yetus-minimaven-plugin in the source 
|      |                |            | passed.
|  +1  |          unit  |   0m  3s   | yetus-assemblies in the source passed. 
|  +1  |          unit  |   0m  4s   | audience-annotations-jdiff in the source 
|      |                |            | passed.
|  +1  |          unit  |   0m  4s   | audience-annotations-component in the 
|      |                |            | source passed.
|  +1  |          unit  |   0m  4s   | precommit in the source passed. 
|  +1  |          unit  |   0m  4s   | releasedocmaker in the source passed. 
|  +1  |          unit  |   0m  4s   | shelldocs in the source passed. 
|  +1  |          unit  |   0m  3s   | asf-site-src in the source passed. 
|  +1  |          unit  |   0m  4s   | yetus-dist in the source passed. 
|  +1  |          unit  |   0m  9s   | yetus-maven-plugin in the source passed. 
|  +1  |          unit  |   0m 10s   | root in the source passed. 
|  +1  |    asflicense  |   0m 45s   | The source tree does not generate ASF 
|      |                |            | License warnings.
|      |                |   9m 55s   | 

{code}
Yeah, no, that's not the correct fix.

I think I'm just going to patch the skipper code for findbugs.

> maven + full build should only use UNION for mvninstall
> -------------------------------------------------------
>
>                 Key: YETUS-809
>                 URL: https://issues.apache.org/jira/browse/YETUS-809
>             Project: Yetus
>          Issue Type: Bug
>          Components: Test Patch
>    Affects Versions: 0.9.0
>            Reporter: Allen Wittenauer
>            Assignee: Allen Wittenauer
>            Priority: Blocker
>             Fix For: 0.10.0
>
>         Attachments: YETUS-809.00.patch
>
>
> In current Yetus master, test-patch stopped reporting findbugs issues when 
> run in qbt mode against the Yetus source tree (there is currently one).  
> Individual patch testing does work, however.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to