Page MenuHomePhabricator

T5423, "is newly created" herald rule fails on dry runs
ClosedPublic

Authored by lpriestley on Aug 7 2014, 11:59 PM.
Tags
None
Referenced Files
F14012691: D10187.id24505.diff
Fri, Nov 1, 3:09 PM
F14009397: D10187.id24505.diff
Wed, Oct 30, 4:12 PM
F14007573: D10187.id24505.diff
Tue, Oct 29, 8:13 AM
F13994033: D10187.diff
Wed, Oct 23, 3:16 AM
F13993560: D10187.id.diff
Tue, Oct 22, 11:43 PM
F13992693: D10187.id24505.diff
Tue, Oct 22, 6:13 PM
F13992690: D10187.id24505.diff
Tue, Oct 22, 6:12 PM
F13992292: D10187.id24504.diff
Tue, Oct 22, 3:39 PM
Subscribers
Tokens
"Piece of Eight" token, awarded by epriestley.

Details

Summary

Fixes T5423, "is newly created" herald rule fails on dry runs

Test Plan

Create herald "is newly created" rule, and do a dry run on an existing pholio mock, differential commit, or maniphest task. Should not return an exception.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

lpriestley retitled this revision from to T5423, "is newly created" herald rule fails on dry runs.
lpriestley updated this object.
lpriestley edited the test plan for this revision. (Show Details)
lpriestley added a reviewer: epriestley.
epriestley awarded a token.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Aug 8 2014, 12:04 AM
epriestley updated this revision to Diff 24505.

Closed by commit rP7204f9fec268 (authored by @lpriestley, committed by @epriestley).