Add `rulePHID` to HeraldEffect
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F15515005: D7781.diff
	Fri, Apr 18, 7:17 AM

	F15514525: D7781.id17607.diff
	Fri, Apr 18, 3:25 AM

	F15511691: D7781.id.diff
	Thu, Apr 17, 4:37 AM

	F15510985: D7781.diff
	Wed, Apr 16, 10:56 PM

	F15505868: D7781.id.diff
	Tue, Apr 15, 6:35 AM

	F15429723: D7781.id17607.diff
	Mon, Mar 24, 3:52 AM

	F15401847: D7781.id17604.diff
	Mar 17 2025, 7:28 PM

	F15395214: D7781.id17607.diff
	Mar 16 2025, 3:49 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