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

Do not trigger Notify twice from TestResultsFinisher #900

Merged
merged 2 commits into from
Nov 20, 2024
Merged

Do not trigger Notify twice from TestResultsFinisher #900

merged 2 commits into from
Nov 20, 2024

Commits on Nov 18, 2024

  1. Do not trigger Notify twice from TestResultsFinisher 

    This also fixes up the return value in case the test results comment was updates.

Plus, it does a little formatting and typing cleanup.
    @Swatinem
    Swatinem committed Nov 18, 2024
    Configuration menu
    Copy the full SHA
    ff1cfc5 View commit details
    Browse the repository at this point in the history

  2. aaaaand fix the tests

    @Swatinem
    Swatinem committed Nov 18, 2024
    Configuration menu
    Copy the full SHA
    bc5307f View commit details
    Browse the repository at this point in the history