Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve MAVEN build Performance #52

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

hongdongni
Copy link

That report generation takes time, slowing down the overall build. Reports are definitely useful, but do you need them every time you run the build. We can conditionally disable generating test reports by setting <disableXmlReport>true<disableXmlReport>. If you need to generate reports, just add -DcloseTestReports=false to the end of mvn build command.

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

@radsz
Copy link
Owner

radsz commented May 20, 2023

Hi,

Technically speaking I see nothing wrong with your PR.

Unfortunately, as explained in the Contribution section of README I can not accept it, unless you also make an explicit agreement to the requirements we have for any contributor.

Thank you for your understanding.

best,
Radek

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