Page MenuHomePhabricator

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

Authored by epriestley on Jul 10 2020, 4:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Dec 23, 11:02 PM
Unknown Object (File)
Mon, Dec 23, 1:12 PM
Unknown Object (File)
Fri, Dec 20, 9:41 PM
Unknown Object (File)
Tue, Dec 17, 6:32 PM
Unknown Object (File)
Thu, Dec 5, 4:38 AM
Unknown Object (File)
Wed, Dec 4, 9:27 AM
Unknown Object (File)
Nov 19 2024, 6:07 PM
Unknown Object (File)
Nov 17 2024, 1:24 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.