Paths

Table of Contentst

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

Authored by epriestley on Feb 3 2017, 2:18 PM.

Tags

None

Referenced Files

	F15561079: D17306.id41621.diff
	Tue, Apr 29, 6:01 PM

	F15559366: D17306.id41618.diff
	Tue, Apr 29, 8:25 AM

	F15557567: D17306.id41618.diff
	Mon, Apr 28, 10:11 PM

	F15541377: D17306.id.diff
	Fri, Apr 25, 11:33 AM

	F15531128: D17306.diff
	Wed, Apr 23, 9:47 AM

	F15447628: D17306.id41621.diff
	Mar 28 2025, 12:43 AM

	F15447627: D17306.id41618.diff
	Mar 28 2025, 12:43 AM

	F15447626: D17306.id.diff
	Mar 28 2025, 12:43 AM

View All Files

Subscribers

None

Details

Reviewers

chad

Maniphest Tasks

T12197: Audits are no longer triggered using singular "Auditor:"

Commits

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

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

Event Timeline

epriestley created this revision.Feb 3 2017, 2:18 PM

Harbormaster completed remote builds in B15515: Diff 41618.Feb 3 2017, 2:19 PM

epriestley mentioned this in T12197: Audits are no longer triggered using singular "Auditor:".Feb 3 2017, 2:31 PM

chad accepted this revision.Feb 3 2017, 4:48 PM

This revision is now accepted and ready to land.Feb 3 2017, 4:48 PM

Closed by commit rPfd0591e16880: Restore "Auditor" as an alias for the commit message field "Auditors" (authored by epriestley, committed by epriestley). · Explain WhyFeb 3 2017, 5:14 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

field/

DifferentialAuditorsCommitMessageField.php

6 lines

Diff 41621

View Options

src/applications/differential/field/DifferentialAuditorsCommitMessageField.php

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