From 249f3a80fe77b22b19648ca43c8d266090a8d662 Mon Sep 17 00:00:00 2001 From: Alex Vandiver Date: Mon, 11 Dec 2017 17:36:18 -0800 Subject: [PATCH] Default the prompt for "Amend HEAD with these patches" to true Summary: Most users, if they have gone through the trouble of accepting the auto-fixes, are most likely going to want to take those changes and attempt to land with them. Assuming "Y" for this prompt streamlines for the more likely flow. Test Plan: `arc lint` Reviewers: #blessed_reviewers, epriestley Reviewed By: #blessed_reviewers, epriestley Subscribers: Korvin, epriestley Differential Revision: https://secure.phabricator.com/D18824 --- src/workflow/ArcanistLintWorkflow.php | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/workflow/ArcanistLintWorkflow.php b/src/workflow/ArcanistLintWorkflow.php index de215aad..75277d4b 100644 --- a/src/workflow/ArcanistLintWorkflow.php +++ b/src/workflow/ArcanistLintWorkflow.php @@ -546,7 +546,8 @@ EOTEXT pht('Automatically amending HEAD with lint patches.')); $amend = true; } else { - $amend = phutil_console_confirm(pht('Amend HEAD with lint patches?')); + $amend = phutil_console_confirm(pht('Amend HEAD with lint patches?'), + false); } if ($amend) {