Page MenuHomePhabricator

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

Authored by epriestley on May 31 2019, 1:18 PM.
Tags
None
Referenced Files
F15455153: D20565.id49063.diff
Sat, Mar 29, 10:33 PM
F15451523: D20565.id49058.diff
Fri, Mar 28, 10:32 PM
F15444030: D20565.id.diff
Thu, Mar 27, 7:25 AM
F15441836: D20565.diff
Wed, Mar 26, 8:17 PM
F15416393: D20565.id49063.diff
Thu, Mar 20, 10:51 AM
F15404514: D20565.id.diff
Tue, Mar 18, 8:04 AM
Unknown Object (File)
Mar 3 2025, 11:46 PM
Unknown Object (File)
Feb 28 2025, 1:04 AM
Subscribers
None

Details

Summary

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

Repository
rP Phabricator
Branch
publish1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 22933
Build 31464: Run Core Tests
Build 31463: arc lint + arc unit