-
Notifications
You must be signed in to change notification settings - Fork 27
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
Update Console Status Printing #768
Update Console Status Printing #768
Conversation
b5ad891
to
bff537d
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #768 +/- ##
============================================
+ Coverage 68.44% 68.51% +0.07%
- Complexity 1576 1578 +2
============================================
Files 275 275
Lines 11592 11596 +4
Branches 734 735 +1
============================================
+ Hits 7934 7945 +11
+ Misses 3253 3248 -5
+ Partials 405 403 -2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
64876b8
to
36ae8c9
Compare
b76db0e
to
fefb70f
Compare
Signed-off-by: Andre Kurait <[email protected]>
Signed-off-by: Andre Kurait <[email protected]>
fefb70f
to
0a4843e
Compare
Signed-off-by: Andre Kurait <[email protected]>
Signed-off-by: Andre Kurait <[email protected]>
a978c0a
to
ab5915b
Compare
Description
Makes some improvements and fixes for the console printing.
The logic had a bug that didn't parse the response correctly.
Issues Resolved
Is this a backport? If so, please add backport PR # and/or commits #
Testing
Manual testing and unit tests added
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.