chadtmiller.com/knotless
git that lives in your Atmosphere account
1
10
A pull's timeline says when each of its commits was committed
Merged Approved
This drops Committed's bare prop and has CommitEvent show each commit's time. I searched web/src and found no other bare caller of Committed, so nothing is left passing the removed prop. The template now renders the committed-short span unconditionally, which is what the commits page does. The edit to the doc comment matches the code. I read both changed files and the diff. I did not run the shots check.
All checks have passed
8 passed, 2 skipped
Merged as 891adcf
A pull's timeline now says when each commit in a run was committed, not only when the last one was: "Bob Tanaka committed 2 hours ago" on every row, the exact time on hover. Pages only; a quick look is enough.
Claude Opus 5.5 · low
What changes
Each commit in a pull's run of commits shows its commit time, as the commits page's rows do. The run's heading still gives the last one's.
Committedloses itsbaremode, which onlyCommitEventused to drop the times; a commit that was written on another day, or committed by someone else, still cuts to "committed <when>" in a list, as on the commits page.Why
A run of six commits read "committed" six times with no times, so there was no way to see when any but the last landed.
How it was checked
just look pull-open,pull-queuedandpull-ejectedwith--element .pushed, desktop light and phone dark: each row reads "<author> committed last week", nothing runs past the edge.Shots-moved: pull-open, pull-queued, pull-ejected
Shortcomings
Relative times in a run often repeat ("2 hours ago" on several rows); the exact time is on hover, as everywhere else.