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.

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.