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

Show security heading only if warnings exist #537

Merged
merged 1 commit into from
Mar 3, 2023
Merged

Show security heading only if warnings exist #537

merged 1 commit into from
Mar 3, 2023

Commits on Mar 3, 2023

  1. Skip security warnings heading if no warnings 

    Test highlights that there is ambiguity in the tested code because there
are two booleans to represent a single condition.  It is possible to
have hideWarning == true and showWarning == true, even though we can
only do one or the other but not both.

Another pull request or another change in this pull request will need
to resolve that issue in the most recently merged pull request.
    @MarkEWaite
    MarkEWaite committed Mar 3, 2023
    Configuration menu
    Copy the full SHA
    6fccb2a View commit details
    Browse the repository at this point in the history