Paths

Table of Contentst

Differential D20539

Add a "{src ...}" Remarkup rule to provide a more flexible way to reference source files in Diffusion
ClosedPublic
Actions

Authored by epriestley on May 21 2019, 7:01 PM.

Tags

None

Referenced Files

	F13281115: D20539.diff
	Sun, Jun 2, 9:45 AM

	F13279609: D20539.id.diff
	Sat, Jun 1, 9:24 PM

	F13268178: D20539.diff
	Wed, May 29, 4:44 AM

	F13258182: D20539.diff
	Sun, May 26, 7:19 AM

	F13253681: D20539.diff
	Sat, May 25, 2:51 AM

	F13237334: D20539.diff
	Tue, May 21, 1:38 PM

	F13236865: D20539.diff
	Tue, May 21, 11:18 AM

	F13213416: D20539.diff
	Fri, May 17, 7:44 AM

Subscribers

Tokens

"Mountain of Wealth" token, awarded by 20after4.

Details

Reviewers

Maniphest Tasks

T13291: Implement a "{src ...}" rule in Remarkup for referencing source files in Diffusion

Commits

rP4180b337cf7e: Add a "{src ...}" Remarkup rule to provide a more flexible way to reference…

Summary

Depends on D20538. Ref T13291. We now recognize full source URIs, but encoding full URIs isn't super human-friendly and we can't do stuff like relative links with them. Add {src ...} as a way to get to this behavior that supports options and more flexible syntax.

Test Plan

Screen Shot 2019-05-21 at 11.55.47 AM.png (590×659 px, 52 KB)

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.May 21 2019, 7:01 PM

Harbormaster completed remote builds in B22846: Diff 48970.May 21 2019, 7:02 PM

epriestley requested review of this revision.May 21 2019, 7:02 PM

amckinley accepted this revision.May 21 2019, 7:22 PM

This revision is now accepted and ready to land.May 21 2019, 7:22 PM

Closed by commit rP4180b337cf7e: Add a "{src ...}" Remarkup rule to provide a more flexible way to reference… (authored by epriestley). · Explain WhyMay 21 2019, 8:12 PM

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rP4180b337cf7e: Add a "{src ...}" Remarkup rule to provide a more flexible way to reference….

I've been wishing for this literally forever. If I could award more mountains of wealth tokens, I most certainly would!

Revision Contents
Changeset List

Path

Size

src/

__phutil_library_map__.php

applications/

diffusion/

application/

PhabricatorDiffusionApplication.php

1 line

remarkup/

DiffusionSourceLinkRemarkupRule.php

221 lines

Diff 48983

src/__phutil_library_map__.php

Loading...

src/applications/diffusion/application/PhabricatorDiffusionApplication.php

Loading...

src/applications/diffusion/remarkup/DiffusionSourceLinkRemarkupRule.php

Loading...