Page MenuHomePhabricator

Implement "words of power" against ApplicationTransactions
ClosedPublic

Authored by epriestley on Feb 28 2014, 12:50 AM.
Tags
None
Referenced Files
F14062482: D8369.diff
Mon, Nov 18, 12:32 PM
F14048056: D8369.diff
Thu, Nov 14, 6:40 AM
F14034285: D8369.diff
Sat, Nov 9, 11:26 PM
F14018698: D8369.diff
Tue, Nov 5, 6:43 PM
F14006577: D8369.id.diff
Mon, Oct 28, 1:19 PM
F13985588: D8369.diff
Sun, Oct 20, 9:34 PM
F13975606: D8369.diff
Oct 18 2024, 10:55 AM
F13967416: D8369.diff
Oct 16 2024, 1:29 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. <_<