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

Add maven parameter to control which reports to generate #1175

Merged
merged 9 commits into from May 1, 2021

Conversation

troosan
Copy link
Contributor

@troosan troosan commented Apr 12, 2021

In some cases we do not need to generate all the report types. For instance when the report picked up by Sonar.
This pull request adds a new optional parameter that can be passed to the maven plugin configuration to control which reports should be generated.

Fixes #9

@marchof marchof self-assigned this Apr 16, 2021
@marchof marchof added this to Implementation in Current work items via automation Apr 16, 2021
@marchof marchof force-pushed the customise-reports-to-generate branch from 7f1d707 to 4d65fc9 Compare April 16, 2021 08:59
@marchof
Copy link
Member

marchof commented Apr 16, 2021

This also fixes #9.

@marchof marchof requested review from marchof and Godin and removed request for marchof April 16, 2021 16:08
@marchof marchof moved this from Implementation to Review in Current work items Apr 16, 2021
@troosan
Copy link
Contributor Author

troosan commented Apr 16, 2021

@marchof thanks for the updates !

@marchof
Copy link
Member

marchof commented Apr 16, 2021

@troosan Sure, thanks for picking up on this long outstanding feature request! I added your Github handle in the change log and file header. If you prefer your full name for attribution please feel free to add it. Also if you seen any issues with my updates please feel free to comment on it.

@Godin Godin added this to the 0.8.7 milestone May 1, 2021
@Godin Godin merged commit acce036 into jacoco:master May 1, 2021
Current work items automation moved this from Review to Done May 1, 2021
@marchof
Copy link
Member

marchof commented May 1, 2021

Thanks @Godin !

@troosan
Copy link
Contributor Author

troosan commented May 1, 2021

Thank you all @marchof @Godin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

Report format selection
3 participants