-
Notifications
You must be signed in to change notification settings - Fork 29.1k
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
test_runner: replace ansi clear with ansi reset #55013
base: main
Are you sure you want to change the base?
Conversation
Review requested:
|
Not for this PR, but is |
I'm going to check if it's used elsewhere, but yes, it's definitely risky |
Great work @pmarchini! LGTM |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #55013 +/- ##
=======================================
Coverage 88.04% 88.04%
=======================================
Files 652 652
Lines 183765 183766 +1
Branches 35863 35862 -1
=======================================
+ Hits 161789 161800 +11
- Misses 15229 15231 +2
+ Partials 6747 6735 -12
|
Fixes: #55009