Page MenuHomePhabricator

Create revisions into "Draft", publish them when builds finish
ClosedPublic

Authored by epriestley on Sep 18 2017, 11:01 PM.
Tags
None
Referenced Files
F19012345: D18628.diff
Nov 22 2025, 12:31 PM
F18919619: D18628.id44740.diff
Nov 9 2025, 12:07 PM
F18913080: D18628.id44740.diff
Nov 9 2025, 5:31 AM
F18786012: D18628.id44723.diff
Oct 14 2025, 4:44 PM
F18717905: D18628.id44740.diff
Sep 29 2025, 2:39 PM
F18648502: D18628.diff
Sep 20 2025, 12:34 PM
F18627856: D18628.diff
Sep 16 2025, 4:57 AM
F18597803: D18628.id.diff
Sep 13 2025, 3:42 AM
Subscribers
None

Details

Summary

Ref T2543. This doesn't stand alone since mail still goes out normally, but gets this piece working: new revisions start as "Draft", then after updates if there are no builds they go into "Needs Review".

This should work in general because builds update revisions when they complete, to publish a "Harbormaster finished build yada yada" transaction. So either we'll un-draft immediately, or un-draft after the last build finishes.

I'll hold this until the mail and some other stuff (like UI hints) are in slightly better shape since I think it's probably too rough on its own.

Test Plan

Created revisions locally, saw them un-draft after builds.

Diff Detail

Repository
rP Phabricator
Branch
draft4
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 18499
Build 24913: Run Core Tests
Build 24912: arc lint + arc unit