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

Fix coverage runfiles directory issue #15031

Closed
wants to merge 1 commit into from
Closed

Fix coverage runfiles directory issue #15031

wants to merge 1 commit into from

Commits on Mar 12, 2022

  1. Fix coverage runfiles directory issue 

    As discovered in bazelbuild#15030 there
was another bug with the short circuiting coverage logic where tests
were not run from the correct directory. In that PR this issue is fixed
directly, with this change we instead make missing LCOV_MERGER be
deferred until after all test setup and run, which I think is more
future proof to other changes here, and also allows users to know their
tests are being run with coverage, and output the correct files, even in
the case they don't setup the coverage merger infrastructure.
    @keith
    keith committed Mar 12, 2022
    Configuration menu
    Copy the full SHA
    607809e View commit details
    Browse the repository at this point in the history