Add `rulePHID` to HeraldEffect
ClosedPublic
Actions

Authored by epriestley on Dec 17 2013, 8:55 PM.

Tags

None

Referenced Files

	F19508084: D7781.diff
	Sat, Jan 10, 1:04 AM

	F18941367: D7781.id17604.diff
	Nov 11 2025, 10:02 AM

	F18876123: D7781.diff
	Nov 5 2025, 9:47 PM

	F18851874: D7781.id.diff
	Oct 31 2025, 4:48 AM

	F18847543: D7781.id17607.diff
	Oct 30 2025, 1:06 AM

	F18847328: D7781.diff
	Oct 30 2025, 12:03 AM

	F18768515: D7781.diff
	Oct 8 2025, 4:00 AM

	F18711786: D7781.id.diff
	Sep 29 2025, 4:54 AM

View All Files

Subscribers

aran

Details

Reviewers

btrahan

Maniphest Tasks

T4195: Implement pre-commit Herald rules

Commits

Restricted Diffusion Commit
rPbaa756a027fa: Add `rulePHID` to HeraldEffect

Summary

Ref T4195. Herald rules gained PHIDs only recently, propagate them to HeraldEffect to make some of the hook stuff eaiser.

Test Plan

iiam

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

btrahan accepted this revision.Dec 17 2013, 9:20 PM

Closed by commit rPbaa756a027fa (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

herald/

engine/

HeraldEffect.php

22 lines

HeraldEngine.php

1 line

Diff 17607

View Options

src/applications/herald/engine/HeraldEffect.php