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.

Sign up for GitHub

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

Jump to bottom

Remove LCOV merger dependency of cc_test without coverage #16994

Closed
wants to merge 1 commit into from
Closed

Remove LCOV merger dependency of cc_test without coverage #16994

wants to merge 1 commit into from

Commits on Dec 12, 2022

  1. Remove LCOV merger dependency of cc_test without coverage 

    When coverage is disabled, `cc_test` should not depend on the Java LCOV
merger tool. This is achieved by using `configuration_field`.

Also depend on coverage tools in `exec` rather than `target`
configuration, matching Java rules.
    @fmeum
    fmeum committed Dec 12, 2022
    Configuration menu
    Copy the full SHA
    4f8b579 View commit details
    Browse the repository at this point in the history