Skip to content

Add baseline_coverage_files to ctx.instrumented_files_info #26133

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

fmeum
Copy link
Collaborator

@fmeum fmeum commented May 22, 2025

Work towards #5716

RELNOTES: Rules can now register their own baseline coverage files in LCOV format via the new baseline_coverage_files parameter of ctx.instrumented_files_info. If the target matches the instrumentation filter, Bazel will merge the data into the combined coverage report generated with --combined_report.

@fmeum fmeum marked this pull request as ready for review May 23, 2025 07:19
@fmeum fmeum requested review from a team, gregestren and fweikert as code owners May 23, 2025 07:19
@fmeum fmeum requested a review from c-mita May 23, 2025 07:19
@github-actions github-actions bot added team-Configurability platforms, toolchains, cquery, select(), config transitions team-Documentation Documentation improvements that cannot be directly linked to other team labels awaiting-review PR is awaiting review from an assigned reviewer labels May 23, 2025
Copy link
Member

@c-mita c-mita left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-review PR is awaiting review from an assigned reviewer team-Configurability platforms, toolchains, cquery, select(), config transitions team-Documentation Documentation improvements that cannot be directly linked to other team labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants