Closed andrewhop closed 2 months ago
Attention: Patch coverage is 62.06897%
with 11 lines
in your changes missing coverage. Please review.
Project coverage is 78.33%. Comparing base (
51890da
) to head (0ba831d
). Report is 17 commits behind head on main.
Files | Patch % | Lines |
---|---|---|
tool-openssl/x509_test.cc | 0.00% | 8 Missing :warning: |
tool-openssl/x509.cc | 85.71% | 3 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Description of changes:
Previously only some output followed the user's request using
-out
to direct the output to a file. This change updates everything to use a commonBIO
which isstdout
by default or the user provided file.Testing:
Added new tests that test writing to a file directly with
-out
we have existing tests that cover writing tostdout
(and then getting redirected to a file with>
) works as expected.By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and the ISC license.