Skip to content
This repository has been archived by the owner on Nov 19, 2024. It is now read-only.

Improve MAVEN build Performance #161

Merged
merged 2 commits into from
Aug 24, 2021
Merged

Improve MAVEN build Performance #161

merged 2 commits into from
Aug 24, 2021

Conversation

ChenZhangg
Copy link
Contributor

Apache Maven Dependency Plugin can be used to find unused dependencies. And I found following list. Maybe we can remove them.
jacoco
{groupId='org.hamcrest', artifactId='hamcrest-core'}
{groupId='org.jenkins-ci.plugins', artifactId='structs'}
{groupId='org.jenkins-ci.main', artifactId='jenkins-war'}
{groupId='com.github.spotbugs', artifactId='spotbugs-annotations'}
{groupId='net.jcip', artifactId='jcip-annotations'}
{groupId='org.codehaus.mojo', artifactId='animal-sniffer-annotations'}
{groupId='commons-logging', artifactId='commons-logging'}
{groupId='org.jenkins-ci', artifactId='test-annotations'}

According to Maven parallel test, we can run tests in parallel.

=====================
If there are any inappropriate modifications in this PR, please give me a reply and I will change them.

@centic9
Copy link
Member

centic9 commented Aug 23, 2021

Thanks for the suggested changes!

It seems some checks fail because of the "parallel" option, maybe we can keep this out of the PR for now to only remove the third-party libraries first so you can then take a closer look why building in parallel is failing.

@ChenZhangg
Copy link
Contributor Author

OK Let me have a try

@centic9 centic9 merged commit 5695383 into jenkinsci:master Aug 24, 2021
@centic9
Copy link
Member

centic9 commented Aug 24, 2021

This looks good now, thanks for the PR!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants