This is an automated email from the ASF dual-hosted git repository. srdo pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/storm.git
The following commit(s) were added to refs/heads/master by this push: new ad3c642 STORM-3414: Remove JavadocMethod checkstyle rule new 70353fc Merge pull request #3027 from krichter722/remove-missing-javadoc ad3c642 is described below commit ad3c64211ff56456056b23fae95db25bdece194e Author: Karl-Philipp Richter <krich...@posteo.de> AuthorDate: Sat Jun 15 20:35:16 2019 +0200 STORM-3414: Remove JavadocMethod checkstyle rule Remove JavadocMethod checkstyle rule because it requires tons of Javadocs to be invented on files which have not been touched for years and whose Javadoc can be covered with clean code. --- storm-checkstyle/src/main/resources/storm/storm_checkstyle.xml | 9 --------- 1 file changed, 9 deletions(-) diff --git a/storm-checkstyle/src/main/resources/storm/storm_checkstyle.xml b/storm-checkstyle/src/main/resources/storm/storm_checkstyle.xml index e7678ce..d1497c8 100644 --- a/storm-checkstyle/src/main/resources/storm/storm_checkstyle.xml +++ b/storm-checkstyle/src/main/resources/storm/storm_checkstyle.xml @@ -244,15 +244,6 @@ <property name="tagOrder" value="@param, @return, @throws, @deprecated"/> <property name="target" value="CLASS_DEF, INTERFACE_DEF, ENUM_DEF, METHOD_DEF, CTOR_DEF, VARIABLE_DEF"/> </module> - <module name="JavadocMethod"> - <property name="scope" value="public"/> - <property name="allowMissingParamTags" value="true"/> - <property name="allowMissingThrowsTags" value="true"/> - <property name="allowMissingReturnTag" value="true"/> - <property name="minLineCount" value="2"/> - <property name="allowedAnnotations" value="Override, Test"/> - <property name="allowThrowsTagsForSubclasses" value="true"/> - </module> <module name="MethodName"> <property name="format" value="^[a-z][a-z0-9][a-zA-Z0-9_]*$"/> <message key="name.invalidPattern"