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

Full support for multi-module projects #49

Closed
5 tasks done
thsaravana opened this issue Jul 23, 2023 · 0 comments · Fixed by #50
Closed
5 tasks done

Full support for multi-module projects #49

thsaravana opened this issue Jul 23, 2023 · 0 comments · Fixed by #50
Assignees
Milestone

Comments

@thsaravana
Copy link
Member

thsaravana commented Jul 23, 2023

For multi-module project we can have two cases:

  1. The jacoco report is consolidated and we have only 1 coverage.xml
  2. Each module has it's own coverage.xml

For both the cases, the jacoco-report action works, but it could be made better:

What is considered as a module?

  • If we have only one coverage.xml file referred in paths, then treat each <group> (if it exists) in the report as a module. If there are no <group>s, then treat it as a single module project.
  • If the paths contains multiple coverage.xml file, then treat each report as a module.
  • If the paths contains multiple coverage.xml file, and each report in turn contains <group>, then treat the report as a module. Either flat out the <group> or treat each <group> as a sub-module.

Things to do:

  • Add capability to segregate coverage info module wise
  • Add rendering capability to create the comment with coverage for modules
    • Show coverage for full project
    • Breakdown coverage module wise
    • Show coverage for files under each module. Have this collapsed by default and users can expand if wanted.
@thsaravana thsaravana added this to the 1.5 milestone Jul 23, 2023
@thsaravana thsaravana self-assigned this Jul 23, 2023
@thsaravana thsaravana added the incubation Features that are under discussion label Jul 23, 2023
@thsaravana thsaravana removed the incubation Features that are under discussion label Jul 24, 2023
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 a pull request may close this issue.

1 participant