From 4c9850278c2182c044e4c09c2d1911c499d17a61 Mon Sep 17 00:00:00 2001 From: Slawek Biel Date: Tue, 22 Mar 2011 12:29:09 -0700 Subject: [PATCH] [Differential: removing reviewed/reviewers check from the parser] Summary: As we've discussed the check is not needed. Test Plan: - php -l Reviewed By: epriestley Reviewers: jungejason, epriestley CC: epriestley Revert Plan: sure Other Notes: Differential Revision: 81 --- .../parser/commitmessage/DifferentialCommitMessage.php | 7 ------- 1 file changed, 7 deletions(-) diff --git a/src/applications/differential/parser/commitmessage/DifferentialCommitMessage.php b/src/applications/differential/parser/commitmessage/DifferentialCommitMessage.php index 42ad2d9d99..43bc42fc64 100755 --- a/src/applications/differential/parser/commitmessage/DifferentialCommitMessage.php +++ b/src/applications/differential/parser/commitmessage/DifferentialCommitMessage.php @@ -403,13 +403,6 @@ class DifferentialCommitMessage { } } - if (isset($fields['Reviewers']) && isset($fields['Reviewed By'])) { - $this->fail( - null, - "Commit message contains both 'Reviewers:' and 'Reviewed By:' ". - "fields."); - } - return $fields; }