Page MenuHomePhabricator

Add "Revision test plan" as a Herald field; remove test plan from the "Revision summary" field
ClosedPublic

Authored by epriestley on Aug 29 2018, 9:15 PM.
Tags
None
Referenced Files
F15489844: D19623.diff
Fri, Apr 11, 12:39 PM
F15460289: D19623.id46907.diff
Mon, Mar 31, 9:32 PM
F15432726: D19623.id.diff
Mon, Mar 24, 7:33 PM
F15426652: D19623.diff
Sun, Mar 23, 10:15 AM
Unknown Object (File)
Feb 26 2025, 8:23 AM
Unknown Object (File)
Feb 26 2025, 6:57 AM
Unknown Object (File)
Feb 25 2025, 3:07 PM
Unknown Object (File)
Feb 9 2025, 6:24 AM
Subscribers
None

Details

Summary

See PHI844. Ref T13189.

Add "Revision test plan" as an available field for Herald. This is a little niche -- and a little odd because it sticks around even if you fully disable test plans -- but probably broadly reasonable.

The existing "Revision summary" field counterintuitively included the test plan. Separate this out since it's now a separate field and the behavior was weird historic nonsense. I'll note this in the changelog.

Test Plan

Wrote a rule using both fields, verified they generated the expected values.

Diff Detail

Repository
rP Phabricator
Branch
herald1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 20712
Build 28154: Run Core Tests
Build 28153: arc lint + arc unit

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Aug 29 2018, 9:17 PM
epriestley requested review of this revision.
This revision was automatically updated to reflect the committed changes.