Page MenuHomePhabricator

Restore "Auditor" as an alias for the commit message field "Auditors"
ClosedPublic

Authored by epriestley on Feb 3 2017, 2:18 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 20 2024, 12:50 PM
Unknown Object (File)
Feb 20 2024, 11:26 AM
Unknown Object (File)
Jan 26 2024, 8:06 AM
Unknown Object (File)
Jan 6 2024, 1:23 AM
Unknown Object (File)
Dec 27 2023, 9:42 AM
Unknown Object (File)
Dec 21 2023, 4:12 PM
Unknown Object (File)
Nov 29 2023, 10:47 PM
Unknown Object (File)
Nov 24 2023, 9:52 AM
Subscribers
None

Details

Summary

Fixes T12197. I think this field was never recognized by Differential (it doesn't appear in D17070, but maybe that isn't the right change).

It was recognized by the ad-hoc regular expression which I replaced with a formal parser in D17262.

Allow the former parser to accept "Auditor" as an alias for "Auditors".

Test Plan

Committed a change with Auditor: dog, saw the audit trigger correctly in the web UI.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable