Page MenuHomePhabricator
Differential D19612

Remove another old remarkup engine callsite in Config
ClosedPublic
Actions

Authored by epriestley on Aug 27 2018, 5:53 PM.
Tags
None
Referenced Files
F13143210: D19612.diff
Fri, May 3, 7:18 AM
F13137454: D19612.diff
Thu, May 2, 7:03 PM
Unknown Object (File)
Wed, May 1, 10:32 PM
Unknown Object (File)
Tue, Apr 23, 6:42 PM
Unknown Object (File)
Fri, Apr 19, 6:05 PM
Unknown Object (File)
Tue, Apr 16, 9:29 AM
Unknown Object (File)
Mon, Apr 15, 6:56 PM
Unknown Object (File)
Thu, Apr 11, 7:13 PM
View All 70 Files
Subscribers
None

Details

Reviewers
amckinley
Maniphest Tasks
T13189: Plans: 2018 Week 35 Bonus Content
Commits
rPee823982a46b: Remove another old remarkup engine callsite in Config
Summary

Ref T13189. Summaries do not appear to be meaningfully rendered with Remarkup so just drop the engine. See D19610 for the previous change in this vein.

Test Plan

Viewed config list with option summaries.

Diff Detail

Repository
rP Phabricator
Branch
config2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 20690
Build 28121: Run Core Tests
Build 28120: arc lint + arc unit

Event Timeline

epriestley created this revision.Aug 27 2018, 5:53 PM
Harbormaster completed remote builds in B20690: Diff 46885.Aug 27 2018, 5:54 PM
epriestley requested review of this revision.Aug 27 2018, 5:54 PM
This revision was not accepted when it landed; it landed in state Needs Review.Aug 27 2018, 6:10 PM
Closed by commit rPee823982a46b: Remove another old remarkup engine callsite in Config (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
config/
controller/
9 lines

Diff 46885

View Options

src/applications/config/controller/PhabricatorConfigGroupController.php

Loading...
Phabricator · Built by Phacility