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
F15578679: D17067.id41055.diff
Tue, May 6, 8:18 PM
F15561140: D17067.id41083.diff
Tue, Apr 29, 6:18 PM
F15550622: D17067.id.diff
Sun, Apr 27, 12:44 PM
F15547446: D17067.diff
Sat, Apr 26, 7:11 PM
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
Apr 5 2025, 2:19 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.