HomePhabricator

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

Authored by epriestley on Fri, May 3, 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
epriestleyTue, May 14, 4:27 PM
Pushed
epriestleyTue, May 14, 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