Page MenuHomePhabricator

Add Differential and Herald mail stamps and some refinements
ClosedPublic

Authored by epriestley on Feb 5 2018, 8:06 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 17, 2:40 PM
Unknown Object (File)
Fri, Apr 12, 4:17 PM
Unknown Object (File)
Sat, Mar 23, 5:01 AM
Unknown Object (File)
Sat, Mar 23, 5:01 AM
Unknown Object (File)
Feb 12 2024, 4:57 AM
Unknown Object (File)
Jan 9 2024, 3:23 PM
Unknown Object (File)
Dec 25 2023, 2:42 PM
Unknown Object (File)
Dec 23 2023, 2:51 AM
Subscribers
None

Details

Summary

Ref T13053. Adds revision stamps (status, reviewers, etc). Adds Herald rule stamps, like the existing X-Herald-Rules header.

Removes the "self" stamps, since you can just write a rule against whatever(@epriestley) equivalently. If there's routing logic around this, it can live in the routing layer. This avoids tons of self-actor, self-mention, self-reviewer, self-blocking-reviewer, self-resigned-reviewer, etc., stamps.

Use natcasesort() instead of sort() so that numeric values (like monograms) sort 9, 80, 700 instead of 700, 80, 9.

Remove the commas from rendering since they don't really add anything.

Test Plan

Edited tasks and revisions, looked at mail stamps, saw stamps that looked pretty reasonable (with no more self stuff, no more commas, sorting numbers, and Herald stamps).

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable