-
Notifications
You must be signed in to change notification settings - Fork 16
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
slabtop: Add command #42
Conversation
Just a hint: instead of using "link to x" you can use one of Github's keywords like "fixes x" to link a PR to an issue. |
Need to introduce the ratatui crate to implement the TUI, so mark pr as draft for the time being |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #42 +/- ##
==========================================
- Coverage 48.37% 45.99% -2.38%
==========================================
Files 15 18 +3
Lines 1933 2187 +254
Branches 277 305 +28
==========================================
+ Hits 935 1006 +71
- Misses 839 1017 +178
- Partials 159 164 +5
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@Krysztal112233 is that still a draft? |
I have a branch that implements simple output, but does not implement the TUI Implementing a TUI was a lot more trouble than I thought it would be |
39f1fe8
to
decffd4
Compare
Fixes "unused manifest key" warnings from Cargo
* w: Format time and get cmdline This change allows w to display time in a similar way to the original procps w. Additionally, w now fetches the cmdline for each entry.
This comment was marked as resolved.
This comment was marked as resolved.
sorry, there is now a small conflict, could you please fix it? should be easy |
the conflict has been fixed |
@sylvestre this pr almost complete, can you have a look about it? |
terrific! |
fixes #31