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.

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

fix: print quick audit report for human output #486

Closed
wants to merge 1 commit into from

Commits on Nov 15, 2019

  1. fix: print quick audit report for human output

    This was broken when the support/funding functionality changed the
    return value to no longer track the promise for the quick audit
    printing.
    
    It was not caught by tests, because they were only running against the
    --json output, and not verifying the quick audit results in any way.
    
    Added a test to track the --json quick audit results (which were not
    broken, but someday could become so) and the human printed quick audit
    results (which were broken).
    
    Paired with @ruyadorno @mikemimik
    isaacs committed Nov 15, 2019
    Copy the full SHA
    639cd3f View commit details
    Browse the repository at this point in the history