Skip to content

"m2e/lifecycle-mapping-metadata.xml" should include "report-aggregate" #427

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

Merged
merged 1 commit into from
Jun 20, 2016

Conversation

Godin
Copy link
Member

@Godin Godin commented Jun 18, 2016

In #388 we forgot to do this.

@Godin Godin added type: bug 🐛 Something isn't working component: maven jacoco-maven-plugin labels Jun 18, 2016
@Godin Godin added this to the 0.7.8 milestone Jun 18, 2016
@Godin Godin self-assigned this Jun 18, 2016
@Godin Godin changed the title "lifecycle-mapping-metadata.xml" should include "report-aggregate" "m2e/lifecycle-mapping-metadata.xml" should include "report-aggregate" Jun 18, 2016
@Godin
Copy link
Member Author

Godin commented Jun 18, 2016

@marchof could you please review?

@marchof marchof merged commit 08a2d0d into master Jun 20, 2016
@marchof marchof deleted the issue-427 branch June 20, 2016 16:50
@marchof
Copy link
Member

marchof commented Jun 20, 2016

@Godin Thanks!

@syashwanthk
Copy link

Hi, Can I please know whether the feature from the change log as "New Maven goal report-aggregate to create reports for multi-module projects (GitHub #388)." help in just aggregating coverage reports of all modules into single one only or also helps in "generating aggregate-report that reports cross module code coverage for the maven plugin"?

@Godin
Copy link
Member Author

Godin commented Sep 28, 2016

@syashwanthk Hi, please use our mailing list ( https://groups.google.com/forum/#!forum/jacoco ) for questions instead of commenting on unrelated tickets. Thank you for your understanding.

Answering your question: it does both.

@syashwanthk
Copy link

Apologies for the same. Thanks for your reply.

@jacoco jacoco locked and limited conversation to collaborators Jan 11, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
component: maven jacoco-maven-plugin type: bug 🐛 Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants