From e225998fce54d84175117f0dc89f625acd48ac36 Mon Sep 17 00:00:00 2001 From: Joshua Spence Date: Tue, 5 May 2015 21:10:25 +1000 Subject: [PATCH] Remove "commit hook mode" check from Javelin linter Summary: Ref T7674. Commit hook mode is going away. Test Plan: Eyeball it. Reviewers: #blessed_reviewers, epriestley Reviewed By: #blessed_reviewers, epriestley Subscribers: Korvin, epriestley Maniphest Tasks: T7674 Differential Revision: https://secure.phabricator.com/D12712 --- src/infrastructure/lint/linter/PhabricatorJavelinLinter.php | 6 ------ 1 file changed, 6 deletions(-) diff --git a/src/infrastructure/lint/linter/PhabricatorJavelinLinter.php b/src/infrastructure/lint/linter/PhabricatorJavelinLinter.php index 4a2298a72a..2cca2030af 100644 --- a/src/infrastructure/lint/linter/PhabricatorJavelinLinter.php +++ b/src/infrastructure/lint/linter/PhabricatorJavelinLinter.php @@ -138,12 +138,6 @@ final class PhabricatorJavelinLinter extends ArcanistLinter { } } - if ($this->getEngine()->getCommitHookMode()) { - // Don't do the dependency checks in commit-hook mode because we won't - // have an available working copy. - return; - } - $external_classes = array(); foreach ($uses as $symbol => $line) { $parts = explode('.', $symbol);