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

JacocoRunner script: update for Bazel 6.0+ #1516

Merged
merged 2 commits into from
Oct 4, 2023
Merged

JacocoRunner script: update for Bazel 6.0+ #1516

merged 2 commits into from
Oct 4, 2023

Commits on Sep 28, 2023

  1. JacocoRunner script: update for Bazel 6.0+ 

    Added a parameter to specify the Bazel major version (to have a single
script for Bazel 5 and 6).
Manually tested on a repo with both Bazel versions.

Had to add a new patch file for Bazel 6.
Added new Bazel branch with the Jacoco interface implementation, updated
for Bazel 6.3.2.

Updated manual test_coverage_with_local_jacocorunner to run the script
with Bazel 6 provided as a parameter (it was failing after the repo's
upgrade to Bazel 6).
    @gergelyfabian
    gergelyfabian committed Sep 28, 2023
    Configuration menu
    Copy the full SHA
    5fa10d7 View commit details
    Browse the repository at this point in the history

  2. JacocoRunner script: renaming scripts to make Bazel 6.0 version the d… 

    …efault

The script for Bazel 5/6 should be the default one.
Leaving the version for Bazel 4 as an alternative with a version suffix
in its filename.
    @gergelyfabian
    gergelyfabian committed Sep 28, 2023
    Configuration menu
    Copy the full SHA
    5918c55 View commit details
    Browse the repository at this point in the history