Page MenuHomePhabricator

Show commit audit status in repository history tables, including merge commit lists
ClosedPublic

Authored by epriestley on Jan 19 2017, 6:53 PM.
Tags
None
Referenced Files
F18906625: D17227.id.diff
Sat, Nov 8, 7:25 PM
F18886466: D17227.diff
Fri, Nov 7, 4:27 AM
F18829210: D17227.diff
Fri, Oct 24, 8:34 PM
F18804304: D17227.id.diff
Oct 18 2025, 7:17 AM
F18784438: D17227.id41438.diff
Oct 13 2025, 12:32 PM
F18784437: D17227.id.diff
Oct 13 2025, 12:32 PM
F18784387: D17227.id41435.diff
Oct 13 2025, 12:26 PM
F18784331: D17227.diff
Oct 13 2025, 12:20 PM
Subscribers
None

Details

Summary

Fixes T6024. Ref T12121. Currently, we show build status in commit history tables; show audit status alongside it.

Also:

  • Change the "Author/Committer" header to just "Author"; I think it's reasonably obvious what "x/y" means (if you can't guess, you can click the commit and likely figure it out) and this gives us a little more space.
  • Make the audit list look more like the corresponding list in Differential, with similar formatting.
Test Plan
  • Viewed history of a repostiory, saw audit status.
  • Viewed a merge commit, saw audit status in the list of merged commits.
  • Viewed a commit search results list.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable