Page MenuHomePhabricator

Implement "words of power" against ApplicationTransactions
ClosedPublic

Authored by epriestley on Feb 28 2014, 12:50 AM.
Tags
None
Referenced Files
F15448393: D8369.id.diff
Fri, Mar 28, 4:35 AM
F15444037: D8369.diff
Thu, Mar 27, 7:26 AM
F15415702: D8369.id19892.diff
Thu, Mar 20, 7:32 AM
F15402867: D8369.id19909.diff
Tue, Mar 18, 12:45 AM
F15400360: D8369.diff
Mon, Mar 17, 11:14 AM
F15336486: D8369.id19909.diff
Sat, Mar 8, 7:05 PM
Unknown Object (File)
Feb 22 2025, 11:53 PM
Unknown Object (File)
Feb 16 2025, 12:51 PM
Subscribers

Details

Summary

Ref T2222. Differential has certain "words of power" (like Ref T123 or Depends on D345) which should expand into a separate transaction when they appear anywhere in text.

Currently, they're respected in only some fields. I'm expanding them to work in any remarkup field, including comments and inline comments.

This partially generalizes transaction expansion/extraction in comments. Eventually, I'll probably implement some very soft sort of reference edge for T4036, maybe.

Test Plan

{F119368}

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

btrahan edited edge metadata.

LGTM, but in your test plan shouldn't D895 also have been attached? (maybe it already was attached or something)

Slightly disappointed "words of power" didn't appear in the diff itself. :/

This revision is now accepted and ready to land.Feb 28 2014, 5:23 PM

I was making those comments on D985 to test that it didn't attach. <_<