Page MenuHomePhabricator

Remove the "authored" subheader from commits
ClosedPublic

Authored by epriestley on Jul 10 2020, 4:20 PM.
Tags
None
Referenced Files
F19811548: D21406.diff
Wed, Mar 4, 1:35 AM
F19788617: D21406.id.diff
Wed, Feb 25, 4:17 PM
F19785830: D21406.diff
Wed, Feb 25, 1:55 AM
F19270785: D21406.id50949.diff
Dec 22 2025, 10:29 PM
F19189923: D21406.diff
Dec 16 2025, 8:48 AM
F19083352: D21406.id51023.diff
Dec 2 2025, 12:34 PM
F18993528: D21406.id50949.diff
Nov 19 2025, 8:10 AM
F18837291: D21406.diff
Oct 27 2025, 4:49 AM
Subscribers
None

Details

Summary

Ref T13552. I'm trying to reduce the number of direct callers to commit authorship metadata. This header seems low-value enough to simply remove; this information is shown more clearly and prominently in the "Provenance" UI.

In particular, commits have multiple dates (authored, committed, pushed) but this header shows only one. It currently shows the author identity and the commit date, which isn't entirely correct. And it potentially uses an "Identity" as a timeline actor, which is conceptually fine but not entirely firm ground.

Test Plan

Viewed a commit, saw no more subheader.

Diff Detail

Repository
rP Phabricator
Branch
publish3
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 24768
Build 34160: Run Core Tests
Build 34159: arc lint + arc unit

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.