-
Notifications
You must be signed in to change notification settings - Fork 34
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
Output redirection & status monitor #207
base: master
Are you sure you want to change the base?
Conversation
Also moved reusable utils to utils package
b0e1f4d
to
e5647fc
Compare
Also adds a status monitor when run in console mode
e5647fc
to
117f6f8
Compare
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #207 +/- ##
==========================================
- Coverage 76.65% 72.62% -4.03%
==========================================
Files 88 104 +16
Lines 9482 10706 +1224
==========================================
+ Hits 7268 7775 +507
- Misses 1968 2649 +681
- Partials 246 282 +36
☔ View full report in Codecov by Sentry. |
I've been thinking about doing that for quite some time as well 😉 |
Sorry didn't wan't to take it away, but it just fit too well into rethinking output handling. |
No problem 👍🏻 |
... I was thinking I'd like to learn how it can be achieved, so I created it mostly from scratch. No external deps but maybe less smart. 256 colors could be supported and there's also an option to read the background on xterm to decide on the theme. Eventually this can be useful but for the moment I try to limit the amount of colors that are used. In the end it shouldn't be annoying. |
I agree. Too many colours is complicated to get right on both a light and dark background anyway. |
Work in progress:
prune
,check
,forget
can be covered in output parsing