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"
