README: refresh the demo GIF and illustrate both views - #3
Merged
Conversation
Replace the hero GIF with today's recording, and swap the two hand-written ASCII mockups under "What you're looking at" for real captures of the process table and the flat profile. The mockups had drifted from the actual output (invented sample counts, a 214-process host, a 16.x postgres path); the captures show what the script really renders.
yeet run resolves tty mode from stdout().is_terminal() when neither --tty nor --no-tty is given, so every documented invocation gets a PTY without the flag. Carrying it in the docs implied it was required. Rewrite the -- placement note to cite --quiet/--watch instead, since it used --tty as its example of a runtime-side flag.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to #2, which merged before this commit landed on the branch. Rebased onto current
main.Changes
assets/hotspot.gifso the existing reference and any external links to that path stay valid. Alt text rewritten: the old one described "a per-frame hover tooltip", but the new recording shows the flame view ofbashtransitioning to the flat table.assets/hotspot-process.giffor the process table,assets/hotspot-flat-profile.giffor the flat profile.Why the mockups had to go
They had drifted from real output: a 214-process host, invented sample counts, and a
postgresql/16path. The actual captures show 54 processes andpostgresql/17. The prose that follows (header / status line / table / legend, and the column table) already described what the captures show, so it needed no edits.Each GIF was checked frame-by-frame against its filename before writing alt text: process table with
dockerdselected; flat profile ofpostgreswith kernel frames tinted purple and the object legend along the bottom. All three are 1222x635;assets/totals 2.6M.Tradeoff worth a look: the ASCII blocks were greppable and readable in plain text, and the GIFs aren't. Happy to restore the fences alongside the images, with the numbers corrected, if you'd rather keep both.
Also verified
Now that the repo is public, I re-ran the one-liner #2 documents, from a clean cache on Debian 13 / yeet 0.21.0:
It clones anonymously, runs
makein yeet's cache (toolchain fetched, no consent prompt), and renders the live process table. The claim in the merged README holds.No em-dashes added.