HomePhabricator

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

Authored by epriestley on May 3 2019, 5:14 PM.

Description

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

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.

Reviewers: amckinley

Reviewed By: amckinley

Differential Revision: https://secure.phabricator.com/D20494

Details

Committed
epriestleyMay 14 2019, 4:27 PM
Pushed
epriestleyMay 14 2019, 4:27 PM
Reviewer
amckinley
Parents
rARCb61e890a0ca8: Remove unnecessary "," from Pylint version regex
Branches
Unknown
Tags
Unknown
Build Status
Buildable 22794
Build 31270: Run Core Tests