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
Unknown Object (File)
Tue, Jan 28, 2:26 PM
Unknown Object (File)
Fri, Jan 24, 7:59 AM
Unknown Object (File)
Tue, Jan 21, 3:43 PM
Unknown Object (File)
Tue, Jan 21, 12:29 PM
Unknown Object (File)
Tue, Jan 21, 11:08 AM
Unknown Object (File)
Dec 31 2024, 1:06 PM
Unknown Object (File)
Dec 26 2024, 6:32 AM
Unknown Object (File)
Dec 24 2024, 9:43 AM
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
Branch
diff18
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 14943
Build 19584: Run Core Tests
Build 19583: arc lint + arc unit

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.