Page MenuHomePhabricator

Support `M123/456` mock references in Remarkup
ClosedPublic

Authored by epriestley on Jun 15 2014, 7:11 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Feb 3, 11:54 AM
Unknown Object (File)
Sun, Feb 2, 3:44 PM
Unknown Object (File)
Wed, Jan 29, 9:37 AM
Unknown Object (File)
Wed, Jan 29, 9:33 AM
Unknown Object (File)
Thu, Jan 23, 6:16 PM
Unknown Object (File)
Tue, Jan 21, 9:13 AM
Unknown Object (File)
Sun, Jan 19, 1:12 PM
Unknown Object (File)
Fri, Jan 17, 11:29 AM
Subscribers

Details

Summary

Fixes T4729. This form is a little fluff, but we show it in the URI when you click an anchor on the page, and doing so seems desirable. I think it's reasonable to support this form, given that it appears in the URI.

Test Plan

Wrote some stuff like M60, M60/71, M60/72/, M60/73/#13 and saw it all get picked up and rendered/linked properly.

Diff Detail

Repository
rP Phabricator
Branch
pho6
Lint
Lint Passed
Unit
Test Failures
Build Status
Buildable 1120
Build 1120: [Placeholder Plan] Wait for 30 Seconds

Unit TestsFailed

TimeTest
0 mstestEverythingImplemented
0 mstestApplicationsInstalled
0 mstestMySQLAgreesWithUsAboutBMP
0 mstestRejectMySQLBMPQueries

Event Timeline

epriestley retitled this revision from to Support `M123/456` mock references in Remarkup.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Jun 15 2014, 7:53 PM
epriestley updated this revision to Diff 22916.

Closed by commit rP99e354952241 (authored by @epriestley).