Permalink
Browse files

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
  • Loading branch information...
1 parent f4c80a1 commit 249f3a80fe77b22b19648ca43c8d266090a8d662 @alexmv alexmv committed Dec 12, 2017
Showing with 2 additions and 1 deletion.
  1. +2 −1 src/workflow/ArcanistLintWorkflow.php
@@ -546,7 +546,8 @@ public function run() {
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) {

0 comments on commit 249f3a8

Please sign in to comment.