HomePhabricator

Consistently use `phutil_console_confirm()`.

Description

Consistently use phutil_console_confirm().

Summary:
PhutilConsole->confirm() is vestigial as noted in efcd70c, as
PhutilConsole may be deprecated sometime in the future. PhutilConsole
was developed as a tool for T4281 but the feature has been removed
since.

Replace existing occurences of the PhutilConsole->confirm() pattern with
phutil_console_confirm(). There should be no change in functionality
since the two functions are interchangeable.

Test Plan: Manually tested by running bin/arc lint, bin/arc diff --preview, bin/arc land --preview.

Reviewers: Blessed Reviewers, epriestley

Reviewed By: Blessed Reviewers, epriestley

Subscribers: epriestley, alexmv

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

Details

Provenance
stefrenAuthored on Jul 6 2017, 2:09 AM
stefrenPushed on Jul 6 2017, 11:27 PM
Reviewer
Blessed Reviewers
Differential Revision
D18185: Consistently use `phutil_console_confirm()`.
Parents
rARCc04f141ab023: Remove obsolete "arc land" flags: --update-with-merge/rebase, --delete-remote
Branches
Unknown
Tags
Unknown
Build Status
Buildable 17633
Build 23669: Run Core Tests