Page MenuHomePhabricator
Differential D21565

Provide a more structured result log for Herald rules
ClosedPublic
Actions

Authored by epriestley on Thu, Feb 18, 6:53 PM.

Details

Reviewers
None
Maniphest Tasks
T13586: In Herald transcripts, surface exceptions encountered while evaluating fields
Commits
rP5bfd6bda7733: Provide a more structured result log for Herald rules
Summary

Ref T13586. In the footsteps of D21563, make Herald rule results more formal and structured to support meaningful exception reporting.

Test Plan

Ran various Herald rules and viewed transcripts, including rules with recursive dependencies and condition exceptions.

Diff Detail

Repository
rP Phabricator
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Thu, Feb 18, 6:53 PM
Harbormaster completed remote builds in B25145: Diff 51336.Thu, Feb 18, 6:54 PM
epriestley requested review of this revision.Thu, Feb 18, 6:54 PM
This revision was not accepted when it landed; it landed in state Needs Review.Fri, Feb 19, 7:16 PM
Closed by commit rP5bfd6bda7733: Provide a more structured result log for Herald rules (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rP5bfd6bda7733: Provide a more structured result log for Herald rules.

Revision Contents
Changeset List

PathSize
src/
applications/
herald/
controller/
48 lines
engine/
272 lines
exception/
3 lines
storage/
transcript/
11 lines
238 lines
69 lines
13 lines

Diff 51350

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/herald/controller/HeraldTranscriptController.php

Loading...
View Options

src/applications/herald/engine/HeraldEngine.php

Loading...
View Options

src/applications/herald/engine/exception/HeraldRuleEvaluationException.php

Loading...
View Options

src/applications/herald/storage/transcript/HeraldConditionResult.php

Loading...
View Options

src/applications/herald/storage/transcript/HeraldRuleResult.php

Loading...
View Options

src/applications/herald/storage/transcript/HeraldRuleTranscript.php

Loading...
View Options

src/applications/herald/storage/transcript/HeraldTranscriptResult.php

Loading...
Phabricator · Built by Phacility