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
F18853586: D19623.id.diff
Fri, Oct 31, 6:07 PM
F18849177: D19623.diff
Thu, Oct 30, 8:12 AM
F18766172: D19623.diff
Oct 7 2025, 3:19 PM
F18749378: D19623.id46908.diff
Oct 4 2025, 3:36 AM
F18696799: D19623.diff
Sep 27 2025, 8:29 AM
F18670700: D19623.id46907.diff
Sep 25 2025, 1:10 AM
F18664007: D19623.diff
Sep 23 2025, 11:24 PM
F18650638: D19623.diff
Sep 21 2025, 1:20 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.