Page MenuHomePhabricator

When a revision is closed, always promote it out of draft

Authored by epriestley on May 31 2019, 1:18 PM.



Fixes T13300. Currently, if you create a revision and then immediately land it (either using --draft or just beating Harbormaster to the punch) it can be stuck in "Draft" forever.

Instead, count landing changes like this as a publishing action.

Test Plan
  • Used arc diff --hold to create a revision, then pushed the commit immediately.
  • Before change: revision closed, but was stuck in draft.
  • After change: revision closed and was promoted out of draft.

Diff Detail

rP Phabricator
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.May 31 2019, 1:18 PM
epriestley requested review of this revision.May 31 2019, 1:19 PM
amckinley accepted this revision.Jun 2 2019, 8:28 AM
This revision is now accepted and ready to land.Jun 2 2019, 8:28 AM