revi-archive/phorge-arcanist
1
0
Fork 0
mirror of https://we.phorge.it/source/arcanist.git synced 2024-11-26 16:52:40 +01:00
Code Issues Releases Wiki Activity

Default the prompt for "Amend HEAD with these patches" to true

Browse source 
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
This commit is contained in:
Alex Vandiver 2017-12-11 17:36:18 -08:00
parent f4c80a114d
commit 249f3a80fe
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/workflow/ArcanistLintWorkflow.php
View file

@ -546,7 +546,8 @@ EOTEXT
pht('Automatically amending HEAD with lint patches.')); pht('Automatically amending HEAD with lint patches.'));
$amend = true; $amend = true;
} else { } else {
$amend = phutil_console_confirm(pht('Amend HEAD with lint patches?')); $amend = phutil_console_confirm(pht('Amend HEAD with lint patches?'),
false);
} }
if ($amend) { if ($amend) {