Page MenuHomePhabricator

If the stars align, make "Land Revision" kind of work
ClosedPublic

Authored by epriestley on Oct 13 2015, 10:36 PM.
Tags
None
Referenced Files
F19055664: D14270.id.diff
Fri, Nov 28, 5:55 PM
F18817809: D14270.id.diff
Oct 21 2025, 6:35 PM
F18817771: D14270.id.diff
Oct 21 2025, 6:17 PM
F18815695: D14270.diff
Oct 21 2025, 3:16 AM
F18814482: D14270.diff
Oct 20 2025, 11:18 PM
F18780879: D14270.id34448.diff
Oct 12 2025, 8:23 PM
F18763793: D14270.diff
Oct 7 2025, 2:16 AM
F18623489: D14270.id.diff
Sep 15 2025, 4:01 PM
Subscribers
None

Details

Summary

Ref T182. If 35 other things are configured completely correctly, make it remotely possible that this button may do something approximating the thing that the user wanted.

This primarily fleshes out the idea that "operations" (like landing, merging or cherry-picking) can have some beahavior, and when we run an operation we do whatever that behavior is instead of just running git show.

Broadly, this isn't too terrible because Drydock seems like it actually works properly for the most part (???!?!).

Test Plan

Screen Shot 2015-10-13 at 3.23.06 PM.png (242×467 px, 28 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to If the stars align, make "Land Revision" kind of work.
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.Oct 13 2015, 10:43 PM
This revision was automatically updated to reflect the committed changes.