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

[devops] Remove logic to detect if the tests failed to build before reporting results. #20715

Merged
merged 3 commits into from
Jun 12, 2024
Merged

[devops] Remove logic to detect if the tests failed to build before reporting results. #20715

merged 3 commits into from
Jun 12, 2024

Commits on Jun 11, 2024

  1. [devops] Remove logic to detect if the tests failed to build before r… 

    …eporting results.

This check is redundant now, because the post-build test pipeline just won't
run in this case.

This fixes an issue where the GitHub comment reports 0 tests passing, because
we'd detect that the tests failed to build even when they didn't.
    @rolfbjarne
    rolfbjarne committed Jun 11, 2024
    1 Configuration menu
    Copy the full SHA
    0525f8d View commit details
    Browse the repository at this point in the history

Commits on Jun 12, 2024

  1. Merge branch 'main' into devops-fix-0-test-results

    @rolfbjarne
    rolfbjarne committed Jun 12, 2024
    Configuration menu
    Copy the full SHA
    38511a6 View commit details
    Browse the repository at this point in the history

  2. [devops] Fix detecting if we're a PR.

    @rolfbjarne
    rolfbjarne committed Jun 12, 2024
    Configuration menu
    Copy the full SHA
    c76916e View commit details
    Browse the repository at this point in the history