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
F17662670: D19287.id46153.diff
Sat, Jul 12, 5:14 PM
F17634367: D19287.diff
Thu, Jul 10, 11:03 PM
Unknown Object (File)
May 31 2025, 4:10 PM
Unknown Object (File)
May 28 2025, 8:00 PM
Unknown Object (File)
May 27 2025, 10:04 PM
Unknown Object (File)
May 6 2025, 3:57 AM
Unknown Object (File)
Apr 30 2025, 4:30 PM
Unknown Object (File)
Apr 27 2025, 7:34 PM
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