-
Notifications
You must be signed in to change notification settings - Fork 521
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
Fix part of #5343: Introduce RunCoverageForTestTarget and CoverageRunner utilities to run code coverage for a specific bazel test target [Blocked: #4886]. #5420
Conversation
@adhiamboperes, before I proceed, could you please confirm if the chaining of the PRs is done right? |
… file content and its related test content in the provided subpackage
Hi @Rd4dev, I'm going to mark this PR as stale because it hasn't had any updates for 7 days. If no further activity occurs within 7 days, it will be automatically closed so that others can take up the issue. |
Explanation
[WIP]
Fixes part of #5343
RunCoverageForTestTarget
was planned to beRunCoverage
, but now since the new approach is to use filenames primarily for the script execution which will need introduction of another script (probably named as RunCoverage for simplicity) to avoid conflicts I renamed this to RunCoverageForTestTarget.Essential Checklist
For UI-specific PRs only
If your PR includes UI-related changes, then: