Page MenuHomePhabricator

Fix a bad getStatus() call which is fataling during Herald rule evaluation
ClosedPublic

Authored by epriestley on Mar 22 2017, 4:48 PM.
Tags
None
Referenced Files
F15460888: D17534.id42171.diff
Tue, Apr 1, 3:11 AM
F15459340: D17534.diff
Mon, Mar 31, 12:04 PM
F15443285: D17534.id42171.diff
Thu, Mar 27, 3:30 AM
F15443137: D17534.id42179.diff
Thu, Mar 27, 2:38 AM
F15442365: D17534.id.diff
Wed, Mar 26, 10:58 PM
F15436018: D17534.id.diff
Tue, Mar 25, 11:25 AM
F15434577: D17534.diff
Tue, Mar 25, 4:32 AM
F15433441: D17534.diff
Mon, Mar 24, 11:11 PM
Subscribers
None

Details

Summary

Hit this while arc diff'ing something which is triggering 2+ rules which add reviewers, I think.

Test Plan

Dug this out of a production stack trace; will push and arc diff again.

Diff Detail

Repository
rP Phabricator
Branch
status1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 16086
Build 21344: Run Core Tests
Build 21343: arc lint + arc unit