Skip to content

Modernize plugin - #41

Open
darxriggs wants to merge 1 commit into
jenkinsci:masterfrom
darxriggs:modernize
Open

Modernize plugin#41
darxriggs wants to merge 1 commit into
jenkinsci:masterfrom
darxriggs:modernize

Conversation

@darxriggs

Copy link
Copy Markdown

Otherwise the build fails at the findbugs Maven goal.

Introducing <dependencyManagement> to set upper bounds for transitive dependencies which have been reported by the maven-enforer-plugin.

Exclude commons-net from jenkins-test-harness and jsr305 from spotbugs-annotations, otherwise they are packaged inside the HPI file for some reason.

Declaring mockito-core with scope test, not the compile default.

The HPI file size gets reduced from 3 MB to 812 KB.

Otherwise the build fails at the "findbugs" Maven goal.

Introducing <dependencyManagement> to set upper bounds for transitive
dependencies which have been reported by the maven-enforer-plugin.

Exclude "commons-net" from "jenkins-test-harness" and "jsr305" from
"spotbugs-annotations", otherwise they are packaged inside the HPI file
for some reason.

Declaring "mockito-core" with scope "test", not the "compile" default.

The HPI file size gets reduced from 3 MB to 812 KB.
@darxriggs

Copy link
Copy Markdown
Author

@pskumar448 Can you have a look at this?

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.

1 participant