Skip to content

add C0/C1 cov statistics - #833

Open
antonkri wants to merge 2 commits into
mainfrom
ankr_add_c0c1_data
Open

antonkri wants to merge 2 commits into
mainfrom
ankr_add_c0c1_data

Conversation

@antonkri

@antonkri antonkri commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

📌 Description

This PR extends Module Verification Report with the functionality to collect code coverage data (after bazel coverage command), aggregate those data per component (:comp) and render this data in the final documentation using module verification report post-template. As the mapping between (sub-)components at the architectural level and source code folders is not clear, and this is needed to know which files belong to which component, module definition (:mod) was extended to provide this additional information. For more information see README.

🚨 Impact Analysis

  • This change does not violate any tool requirements and is covered by existing tool requirements
  • This change does not violate any design decisions
  • Otherwise I have created a ticket for new tool qualification

✅ Checklist

  • Added/updated documentation for new or changed features
  • Added/updated tests to cover the changes
  • Followed project coding standards and guidelines

@antonkri
antonkri marked this pull request as draft September 16, 2026 09:47
@antonkri
antonkri force-pushed the ankr_add_c0c1_data branch 4 times, most recently from c5a23e9 to 2050a3c Compare September 16, 2026 14:36
@antonkri
antonkri marked this pull request as ready for review September 16, 2026 14:54
@MaximilianSoerenPollak

Copy link
Copy Markdown
Contributor

@antonkri There is no description or anything what this PR should acomplish, why it is needed or what it builds towards / upon.

It's a bit hard to review it without that knowledge.
we can review the code, but we still can not judge if this is something we want / should have in DaC.

Could you add this to the PR description please? Thanks.

@AlexanderLanin

Copy link
Copy Markdown
Member

And how it achieves what it does, as its very difficult to infer that from reading the code.

@antonkri

Copy link
Copy Markdown
Contributor Author

@AlexanderLanin

AlexanderLanin commented Sep 17, 2026

Copy link
Copy Markdown
Member

@antonkri thanks that helps a lot!

review in progress (not a request to change anything yet!)

  • review events where this registers
  • find a better solution for source_roots
  • update README
  • use non zipped output from coverage tool
  • warning on not found
  • remove env reading from the extension (happens in parallel PRs for all extensions)
  • ensure this works with score coverage tooling, e.g. set it as bazel dependency?
  • all of this should run on comp and not on mod
  • switch location to execpath
  • according to description this currently doesnt work in bazel build mode

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

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants