Page MenuHomePhabricator
Files F15381113

D21303.diff
No OneTemporary
Actions

D21303.diff
View Options

diff --git a/src/workflow/ArcanistDiffWorkflow.php b/src/workflow/ArcanistDiffWorkflow.php
--- a/src/workflow/ArcanistDiffWorkflow.php
+++ b/src/workflow/ArcanistDiffWorkflow.php
@@ -295,9 +295,6 @@
'skip-staging' => array(
'help' => pht('Do not copy changes to the staging area.'),
),
- 'ignore-unsound-tests' => array(
- 'help' => pht('Ignore unsound test failures without prompting.'),
- ),
'base' => array(
'param' => 'rules',
'help' => pht('Additional rules for determining base revision.'),
@@ -1238,22 +1235,20 @@
pht('No unit test failures.'));
break;
case ArcanistUnitWorkflow::RESULT_UNSOUND:
- if ($this->getArgument('ignore-unsound-tests')) {
- echo phutil_console_format(
- "<bg:yellow>** %s **</bg> %s\n",
- pht('UNIT UNSOUND'),
- pht(
- 'Unit testing raised errors, but all '.
- 'failing tests are unsound.'));
- } else {
- $continue = phutil_console_confirm(
- pht(
- 'Unit test results included failures, but all failing tests '.
- 'are known to be unsound. Ignore unsound test failures?'));
- if (!$continue) {
- throw new ArcanistUserAbortException();
- }
+ $continue = phutil_console_confirm(
+ pht(
+ 'Unit test results included failures, but all failing tests '.
+ 'are known to be unsound. Ignore unsound test failures?'));
+ if (!$continue) {
+ throw new ArcanistUserAbortException();
}
+
+ echo phutil_console_format(
+ "<bg:yellow>** %s **</bg> %s\n",
+ pht('UNIT UNSOUND'),
+ pht(
+ 'Unit testing raised errors, but all '.
+ 'failing tests are unsound.'));
break;
case ArcanistUnitWorkflow::RESULT_FAIL:
$this->console->writeOut(

File Metadata

Mime Type
text/plain
Expires
Sat, Mar 15, 5:57 AM (3 w, 14 h ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7384019
Default Alt Text
D21303.diff (1 KB)

Event Timeline

Phabricator · Built by Phacility