Page MenuHomePhabricator
Differential D21565

Provide a more structured result log for Herald rules
ClosedPublic
Actions

Authored by epriestley on Feb 18 2021, 6:53 PM.
Tags
None
Referenced Files
F18106496: D21565.diff
Sun, Aug 10, 11:06 PM
F18052525: D21565.id51350.diff
Mon, Aug 4, 2:50 AM
F18050025: D21565.id.diff
Sun, Aug 3, 6:12 PM
F17990291: D21565.diff
Sat, Aug 2, 12:25 AM
F17878573: D21565.id51336.diff
Mon, Jul 28, 8:24 PM
F17853999: D21565.diff
Sun, Jul 27, 2:34 PM
F17851014: D21565.id51350.diff
Sun, Jul 27, 10:47 AM
F17756256: D21565.id51350.diff
Jul 22 2025, 1:53 PM
View All Files
Subscribers
None

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.

Screen Shot 2021-02-18 at 10.51.49 AM.png (354×1 px, 61 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 18 2021, 6:53 PM
Harbormaster completed remote builds in B25145: Diff 51336.Feb 18 2021, 6:54 PM
epriestley requested review of this revision.Feb 18 2021, 6:54 PM
This revision was not accepted when it landed; it landed in state Needs Review.Feb 19 2021, 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