Page MenuHomePhabricator

Modernize "Author" and "Committer" rendering for commits
ClosedPublic

Authored by epriestley on Jul 10 2020, 4:16 PM.
Tags
None
Referenced Files
F13169779: D21405.diff
Tue, May 7, 8:36 AM
Unknown Object (File)
Sun, May 5, 3:21 AM
Unknown Object (File)
Fri, May 3, 9:27 PM
Unknown Object (File)
Fri, May 3, 8:38 AM
Unknown Object (File)
Thu, Apr 25, 7:21 AM
Unknown Object (File)
Thu, Apr 25, 2:29 AM
Unknown Object (File)
Sat, Apr 20, 4:33 PM
Unknown Object (File)
Thu, Apr 18, 4:55 PM
Subscribers
None

Details

Summary

Ref T13552. Give "Commit" objects a more modern, identity-aware way to render author and committer information.

This uses handles in a more modern way and gives us a single read callsite for raw author and committer names.

Test Plan
  • Grepped for callers to the old methods, found none. (There are a lot of "renderAuthor()" callers in transactions, but this call takes no parameters.)
  • Viewed some commits, saw sensible lists of authors and committers.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Aug 12 2020, 3:58 PM
This revision was automatically updated to reflect the committed changes.