From bdab422440f5d7417403d23b770d9c2131a45c70 Mon Sep 17 00:00:00 2001 From: Aviv Eyal Date: Tue, 8 Sep 2015 10:39:26 -0700 Subject: [PATCH] arc linters: switch `name` and `short` Summary: I think they were wrong. Test Plan: `arc linters nolint`. It would fail without it. Reviewers: #blessed_reviewers, epriestley Reviewed By: #blessed_reviewers, epriestley Subscribers: epriestley, #lint Differential Revision: https://secure.phabricator.com/D14084 --- src/workflow/ArcanistLintersWorkflow.php | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/workflow/ArcanistLintersWorkflow.php b/src/workflow/ArcanistLintersWorkflow.php index a521a586..3c7d5606 100644 --- a/src/workflow/ArcanistLintersWorkflow.php +++ b/src/workflow/ArcanistLintersWorkflow.php @@ -104,8 +104,8 @@ EOTEXT $console->writeOut( "** %s ** **%s** (%s)\n", $text, - nonempty($linter['short'], '-'), - $linter['name']); + nonempty($linter['name'], '-'), + $linter['short']); if ($linter['exception']) { $console->writeOut( @@ -241,11 +241,11 @@ EOTEXT } $linter_info[$key] = array( - 'short' => $linter->getLinterConfigurationName(), + 'name' => $linter->getLinterConfigurationName(), 'class' => get_class($linter), 'status' => $status, 'version' => $version, - 'name' => $linter->getInfoName(), + 'short' => $linter->getInfoName(), 'uri' => $linter->getInfoURI(), 'description' => $linter->getInfoDescription(), 'exception' => $exception,