Page MenuHomePhabricator

Herald - unleash some recursive madness!
ClosedPublic

Authored by btrahan on Feb 6 2014, 8:34 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 29 2026, 1:16 PM
Unknown Object (File)
Mar 23 2026, 10:36 PM
Unknown Object (File)
Mar 14 2026, 9:27 AM
Unknown Object (File)
Mar 12 2026, 8:27 PM
Unknown Object (File)
Jan 28 2026, 2:15 PM
Unknown Object (File)
Jan 15 2026, 1:59 AM
Unknown Object (File)
Nov 28 2025, 5:28 AM
Unknown Object (File)
Oct 30 2025, 6:42 AM

Details

Summary

turn on herald rules ability to specify other herald rules. Fixes T4294.

Test Plan

made a rule to be cc'd on new tasks. made another rule to flag a task if it contained "test test" in the title AND the cc'd rule for new tasks matched. Made some new tasks and verified new "test test" tasks were flagged.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

(This doesn't give us two "Another Herald Rule" fields in Differential or anything, right?)

btrahan updated this revision to Unknown Object (????).Feb 6 2014, 8:42 PM

tested differential rule and found a typo from a previous diff; fixed it.

removed FIELD_RULE from commit and differential since its now always included. Note no bug before but I think the code is more clear this way.