Page MenuHomePhabricator
Differential D19287

When remote builds fail, demote revisions to "Changes Planned + But, Still A Draft"
ClosedPublic
Actions

Authored by epriestley on Apr 3 2018, 4:28 PM.
Tags
None
Referenced Files
F14083240: D19287.diff
Sat, Nov 23, 3:28 AM
Unknown Object (File)
Thu, Nov 21, 9:33 AM
Unknown Object (File)
Tue, Nov 19, 8:44 AM
Unknown Object (File)
Fri, Nov 15, 8:00 AM
Unknown Object (File)
Mon, Nov 11, 8:33 AM
Unknown Object (File)
Mon, Nov 11, 5:43 AM
Unknown Object (File)
Wed, Nov 6, 5:50 PM
Unknown Object (File)
Tue, Nov 5, 1:32 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13110: Plans: Hefty Differential revisions, draft state transitions, and exotic interactions
Commits
rPadf8fdef0e89: When remote builds fail, demote revisions to "Changes Planned + But, Still A…
Summary

Depends on D19286. Ref T13110. After builds fail remote builds, put revisions back in the author's queue.

This doesn't actually notify the author quite yet.

Test Plan

Made a failing build plan run on revisions, created a revision, saw it demote after builds failed.

Diff Detail

Repository
rP Phabricator
Branch
bengine10
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/applications/differential/editor/DifferentialTransactionEditor.php:1594XHP16TODO Comment
Unit
Tests Passed
Build Status
Buildable 19974
Build 27083: Run Core Tests
Build 27082: arc lint + arc unit

Revision Contents
Changeset List

Diff 46153

View Options

src/applications/differential/editor/DifferentialTransactionEditor.php

Loading...
View Options

src/applications/differential/xaction/DifferentialRevisionActionTransaction.php

Loading...
View Options

src/applications/differential/xaction/DifferentialRevisionPlanChangesTransaction.php

Loading...
Phabricator · Built by Phacility