From ec12b710aac0d7f4d6c3bad0f1a8993e7f2d0f84 Mon Sep 17 00:00:00 2001 From: Joshua Spence Date: Tue, 7 Apr 2015 18:08:49 +1000 Subject: [PATCH] Re-enable the deprecated calls setup check Summary: This reverts commit 55c00ebfa125199e4914e98574950e6111a416d7. T7111, T7112 and T7113 have all been resolved now. Test Plan: Maybe wait a few more weeks... I just wanted to ensure that this wasn't forgotten. Reviewers: btrahan, #blessed_reviewers, epriestley Reviewed By: #blessed_reviewers, epriestley Subscribers: Korvin, epriestley Differential Revision: https://secure.phabricator.com/D11910 --- .../conduit/check/ConduitDeprecatedCallSetupCheck.php | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/applications/conduit/check/ConduitDeprecatedCallSetupCheck.php b/src/applications/conduit/check/ConduitDeprecatedCallSetupCheck.php index ba81002d8b..c7ee2765fd 100644 --- a/src/applications/conduit/check/ConduitDeprecatedCallSetupCheck.php +++ b/src/applications/conduit/check/ConduitDeprecatedCallSetupCheck.php @@ -3,10 +3,6 @@ final class ConduitDeprecatedCallSetupCheck extends PhabricatorSetupCheck { protected function executeChecks() { - // NOTE: We still call deprecated methods, so don't enable this until at - // least 30 days after fixing T7111, T7112 and T7113. - return; - $methods = id(new PhabricatorConduitMethodQuery()) ->setViewer(PhabricatorUser::getOmnipotentUser()) ->withIsDeprecated(true)