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

Enable coverage for goveralls #5583

Merged
merged 3 commits into from
May 12, 2021
Merged

Enable coverage for goveralls #5583

merged 3 commits into from
May 12, 2021

Commits on May 12, 2021

  1. Re-enable unit test coverage for goveralls 

    Enables the race flag in bazel tests. Also adds test.log files output
to artifacts for having more information. Updates the go rules and
gazelle rules for bazel.

Disables unsafeptr nogo rule, as that now
fails, even though it was mentioned in ignored vendor dependencies.

Signed-off-by: Daniel Hiller <dhiller@redhat.com>
    @dhiller
    dhiller committed May 12, 2021
    Configuration menu
    Copy the full SHA
    c7c5bdf View commit details
    Browse the repository at this point in the history

  2. Update WORKSPACE and bazel build files 

    Also update vendor folders

Signed-off-by: Daniel Hiller <dhiller@redhat.com>
    @dhiller
    dhiller committed May 12, 2021
    Configuration menu
    Copy the full SHA
    0d75023 View commit details
    Browse the repository at this point in the history

  3. Reenable unsafeptr nogo rule 

    Add rules_go_work- folder to exclusion rules instead.

Signed-off-by: Daniel Hiller <dhiller@redhat.com>
    @dhiller
    dhiller committed May 12, 2021
    Configuration menu
    Copy the full SHA
    a79ee42 View commit details
    Browse the repository at this point in the history