Page MenuHomePhabricator

Drive CLI-based revision edits through "differential.revision.edit" API + EditEngine
ClosedPublic

Authored by epriestley on Dec 16 2016, 1:46 PM.
Tags
None
Referenced Files
F15518969: D17067.id.diff
Sat, Apr 19, 6:34 PM
F15509976: D17067.id41055.diff
Wed, Apr 16, 5:44 PM
F15505655: D17067.diff
Tue, Apr 15, 4:41 AM
F15470957: D17067.id41083.diff
Sat, Apr 5, 2:19 AM
F15466193: D17067.id41055.diff
Thu, Apr 3, 2:46 AM
F15441589: D17067.diff
Mar 26 2025, 6:52 PM
F15433986: D17067.id41083.diff
Mar 25 2025, 1:26 AM
F15429544: D17067.id41083.diff
Mar 24 2025, 2:54 AM
Subscribers
None

Details

Summary

Ref T11114. This creates differential.revision.edit (a modern, v3 API method) and redefines the existing methods in terms of it.

Both differential.createrevision and differential.updaterevision are now internally implemented by building a differential.revision.edit API call and then executing it.

I think this covers everything except custom fields, which need some tweaking to work with EditEngine. I'll clean that up in the next change.

Test Plan
  • Created, updated, and edited revisions via arc.
  • Called APIs manually via test console.
  • Stored custom fields ("Blame Rev", "Revert Plan") aren't exposed yet.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Drive CLI-based revision edits through "differential.revision.edit" API + EditEngine.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Dec 16 2016, 3:18 PM
This revision was automatically updated to reflect the committed changes.