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)
Tue, Nov 19, 6:07 PM
Unknown Object (File)
Sun, Nov 17, 1:24 PM
Unknown Object (File)
Sat, Nov 16, 3:19 AM
Unknown Object (File)
Tue, Nov 12, 1:24 PM
Unknown Object (File)
Fri, Nov 8, 1:04 PM
Unknown Object (File)
Oct 25 2024, 5:29 AM
Unknown Object (File)
Sep 21 2024, 3:09 AM
Unknown Object (File)
Sep 15 2024, 1:09 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.