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)
Thu, Jun 25, 12:03 PM
Unknown Object (File)
Sat, Jun 20, 10:13 PM
Unknown Object (File)
Jun 7 2026, 10:56 AM
Unknown Object (File)
Jun 5 2026, 4:22 AM
Unknown Object (File)
Mar 7 2026, 11:07 PM
Unknown Object (File)
Mar 7 2026, 1:08 PM
Unknown Object (File)
Jan 29 2026, 10:36 PM
Unknown Object (File)
Jan 19 2026, 5:14 PM
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
Tests Passed
Build Status
Buildable 1122
Build 1122: [Placeholder Plan] Wait for 30 Seconds

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).