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