Skip to content

Conversation

@ndimiduk
Copy link
Member

  • HBASE-20467 Precommit personality should only run checkstyle once if we're going to run it at the root.
  • HBASE-24078 SpotBugs check automatically skip inapplicable modules

HBASE-20467 is a good thing to have for PR builds, and is a prerequisite of the checkstyle cleanup performed in HBASE-24078.

NihalJain and others added 2 commits March 31, 2020 09:34
…we're going to run it at the root

Signed-off-by: Sean Busbey <busbey@apache.org>
Without patch, the spotbugs results include warnings for inapplicable
modules:

+---------------------------------------------------------------------------
|      |                |            | Compile Tests
+---------------------------------------------------------------------------
|   0  |        mvndep  |   0m 11s   | Maven dependency ordering
|  +1  |    mvninstall  |   2m 47s   | the source passed
|   0  |      spotbugs  |   0m 11s   | branch/hbase-annotations no spotbugs
|      |                |            | output file (spotbugsXml.xml)
|   0  |      spotbugs  |   0m 14s   | branch/hbase-assembly no spotbugs output
|      |                |            | file (spotbugsXml.xml)
|   0  |      spotbugs  |   0m 07s   |
|      |                |            | branch/hbase-archetypes/hbase-archetype-bu
|      |                |            | ilder no spotbugs output file
|      |                |            | (spotbugsXml.xml)
+---------------------------------------------------------------------------

With patch,

+---------------------------------------------------------------------------
|      |                |            | Compile Tests
+---------------------------------------------------------------------------
|   0  |        mvndep  |   0m 13s   | Maven dependency ordering
|  +1  |    mvninstall  |   3m 09s   | the source passed
|  +1  |      spotbugs  |  17m 09s   | the source passed
+---------------------------------------------------------------------------

Signed-off-by: stack <stack@apache.org>
@ndimiduk ndimiduk merged commit 4affd65 into apache:branch-2.3 Mar 31, 2020
@ndimiduk ndimiduk deleted the 24078-spotbugs-skip-inapplicable-branch-2.3 branch March 31, 2020 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants