From 1779abef6f5ac301db964055f3ac38533de43d84 Mon Sep 17 00:00:00 2001 From: vrana Date: Thu, 16 Aug 2012 14:27:35 -0700 Subject: [PATCH] Link docs from message Test Plan: phutil_console_confirm("See http://www.phabricator.com/docs/phabricator/article/Differential_User_Guide_Large_Changes.html for information"); Reviewers: epriestley Reviewed By: epriestley CC: aran, Korvin Differential Revision: https://secure.phabricator.com/D3317 --- src/workflow/ArcanistDiffWorkflow.php | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/workflow/ArcanistDiffWorkflow.php b/src/workflow/ArcanistDiffWorkflow.php index 7b92cd37..7d9f292a 100644 --- a/src/workflow/ArcanistDiffWorkflow.php +++ b/src/workflow/ArcanistDiffWorkflow.php @@ -849,11 +849,15 @@ EOTEXT if (count($changes) > 250) { $count = number_format(count($changes)); + $link = + "http://www.phabricator.com/docs/phabricator/article/". + "Differential_User_Guide_Large_Changes.html"; $message = "This diff has a very large number of changes ({$count}). ". "Differential works best for changes which will receive detailed ". "human review, and not as well for large automated changes or ". - "bulk checkins. Continue anyway?"; + "bulk checkins. See {$link} for information about reviewing big ". + "checkins. Continue anyway?"; if (!phutil_console_confirm($message)) { throw new ArcanistUsageException( "Aborted generation of gigantic diff.");