Legoktm has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/327440 )

Change subject: Don't require documentation for test cases
......................................................................

Don't require documentation for test cases

Change-Id: I375ea9c889f40a230e30529f442cd93e7959113d
---
M MediaWiki/Sniffs/Commenting/FunctionCommentSniff.php
1 file changed, 4 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/tools/codesniffer 
refs/changes/40/327440/1

diff --git a/MediaWiki/Sniffs/Commenting/FunctionCommentSniff.php 
b/MediaWiki/Sniffs/Commenting/FunctionCommentSniff.php
index cfe9e74..fa3464f 100644
--- a/MediaWiki/Sniffs/Commenting/FunctionCommentSniff.php
+++ b/MediaWiki/Sniffs/Commenting/FunctionCommentSniff.php
@@ -39,6 +39,10 @@
         * @return void
         */
        public function process( PHP_CodeSniffer_File $phpcsFile, $stackPtr ) {
+               if ( substr( $phpcsFile->getFilename(), -8 ) === 'Test.php' ) {
+                       // Don't check documentation for test cases
+                       return;
+               }
                $tokens = $phpcsFile->getTokens();
                $find   = PHP_CodeSniffer_Tokens::$methodPrefixes;
                $find[] = T_WHITESPACE;

-- 
To view, visit https://gerrit.wikimedia.org/r/327440
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I375ea9c889f40a230e30529f442cd93e7959113d
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/tools/codesniffer
Gerrit-Branch: master
Gerrit-Owner: Legoktm <lego...@member.fsf.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to