Page MenuHomePhabricator

Fix a potential double-prompt in "arc land" when landing with ongoing builds
ClosedPublic

Authored by epriestley on May 3 2019, 5:16 PM.
Tags
None
Referenced Files
F15486169: D20494.id48886.diff
Thu, Apr 10, 5:24 AM
F15479738: D20494.id.diff
Tue, Apr 8, 9:08 AM
F15476065: D20494.diff
Mon, Apr 7, 3:15 AM
F15475517: D20494.id48921.diff
Sun, Apr 6, 8:06 PM
F15464694: D20494.id48886.diff
Wed, Apr 2, 12:26 PM
F15450788: D20494.diff
Fri, Mar 28, 6:15 PM
F15450267: D20494.diff
Fri, Mar 28, 2:46 PM
F15424933: D20494.id48921.diff
Sun, Mar 23, 1:13 AM
Subscribers
None

Details

Reviewers
amckinley
Summary

The recently-added, build-plan-behavior-aware check here does all of its own prompting, so we should skip the other prompting if it doesn't throw.

Test Plan

Will arc land something sketchy sooner or later.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable