Page MenuHomePhabricator

Implement "words of power" against ApplicationTransactions
ClosedPublic

Authored by epriestley on Feb 28 2014, 12:50 AM.
Tags
None
Referenced Files
F14357283: D8369.diff
Fri, Dec 20, 3:46 AM
Unknown Object (File)
Sun, Dec 15, 9:23 AM
Unknown Object (File)
Sat, Dec 14, 7:53 PM
Unknown Object (File)
Tue, Dec 10, 7:32 AM
Unknown Object (File)
Mon, Dec 9, 7:41 PM
Unknown Object (File)
Sat, Dec 7, 9:56 PM
Unknown Object (File)
Wed, Dec 4, 3:50 PM
Unknown Object (File)
Tue, Dec 3, 8:41 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. <_<