Page MenuHomePhabricator

Allow draft revisions to be commandeered
ClosedPublic

Authored by epriestley on Mar 14 2018, 9:01 PM.
Tags
None
Referenced Files
F19030295: D19228.id46039.diff
Mon, Nov 24, 8:46 PM
F19030293: D19228.id46040.diff
Mon, Nov 24, 8:45 PM
F19026507: D19228.diff
Mon, Nov 24, 10:55 AM
F18899876: D19228.id.diff
Fri, Nov 7, 8:18 PM
F18875927: D19228.diff
Wed, Nov 5, 8:17 PM
F18852873: D19228.id46040.diff
Fri, Oct 31, 12:38 PM
F18845593: D19228.id46039.diff
Wed, Oct 29, 1:21 PM
F18769540: D19228.diff
Oct 8 2025, 7:59 AM
Subscribers
None

Details

Summary

See PHI457. There's no real reason not to allow this, it just wasn't clear if it was useful. See D18626.

An install had a user arc diff and then sprint out the door to take a very long vacation before the builds finished. One failed, so the revision is stuck as a draft forever. This seems like a reasonable motivation for allowing "Commandeer".

Test Plan

Successfully commandeered a draft.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Mar 14 2018, 9:04 PM
This revision was automatically updated to reflect the committed changes.