Page MenuHomePhabricator

Move Differential commit message rendering to dedicated classes
ClosedPublic

Authored by epriestley on Dec 16 2016, 12:28 PM.
Tags
None
Referenced Files
F18168454: D17066.id41054.diff
Fri, Aug 15, 4:11 PM
F18047726: D17066.id41082.diff
Sun, Aug 3, 1:06 PM
F17921957: D17066.id.diff
Jul 30 2025, 6:05 AM
F17900484: D17066.diff
Jul 29 2025, 9:13 AM
F17866514: D17066.id41054.diff
Jul 28 2025, 4:41 AM
F17848314: D17066.diff
Jul 27 2025, 7:14 AM
F17741564: D17066.id41082.diff
Jul 21 2025, 9:31 AM
F17737483: D17066.diff
Jul 20 2025, 8:53 PM
Subscribers
None

Details

Summary

Ref T11114. This probably still has some bugs, but survives basic sanity checks.

Continue pulling commit message logic out of CustomField so we can reduce the amount of responsibility/bloat in the classtree and send more code through EditEngine.

Test Plan
  • Called differential.getcommitmessage via API console for various revisions/parameters (edit and create mode, with and without fields, with and without revisions).
  • Used --create, --edit and --update modes of arc diff from the CLI.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Move Differential commit message rendering to dedicated classes.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.

I've also removed the "pro tips" in commit message templates since I don't get the sense that anyone ever actually read them. We could restore them later if they are missed.

chad edited edge metadata.
This revision is now accepted and ready to land.Dec 16 2016, 3:12 PM
This revision was automatically updated to reflect the committed changes.