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
F15554741: D17066.id41082.diff
Mon, Apr 28, 8:19 AM
F15549293: D17066.id.diff
Sun, Apr 27, 4:57 AM
F15545056: D17066.diff
Sat, Apr 26, 8:46 AM
F15453129: D17066.id41054.diff
Mar 29 2025, 9:39 AM
F15449336: D17066.id41082.diff
Mar 28 2025, 9:36 AM
F15443701: D17066.id.diff
Mar 27 2025, 5:44 AM
F15439926: D17066.diff
Mar 26 2025, 10:18 AM
F15406508: D17066.diff
Mar 18 2025, 12:36 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.