[ofbiz-framework] branch trunk updated: Improved: Corrected max parameter number allowed checkstyle rule, also added proeprty for ignoring overridden methods. (OFBIZ-11931) Thanks Jacques for your inputs.

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[ofbiz-framework] branch trunk updated: Improved: Corrected max parameter number allowed checkstyle rule, also added proeprty for ignoring overridden methods. (OFBIZ-11931) Thanks Jacques for your inputs.

surajk
This is an automated email from the ASF dual-hosted git repository.

surajk pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git


The following commit(s) were added to refs/heads/trunk by this push:
     new 6df66ff  Improved: Corrected max parameter number allowed checkstyle rule, also added proeprty for ignoring overridden methods. (OFBIZ-11931) Thanks Jacques for your inputs.
6df66ff is described below

commit 6df66ff7a71136b0975bbf48b28967969df154df
Author: Suraj Khurana <[hidden email]>
AuthorDate: Sun Aug 9 12:43:59 2020 +0530

    Improved: Corrected max parameter number allowed checkstyle rule, also added proeprty for ignoring overridden methods.
    (OFBIZ-11931)
    Thanks Jacques for your inputs.
---
 config/checkstyle/checkstyle.xml | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/config/checkstyle/checkstyle.xml b/config/checkstyle/checkstyle.xml
index 26d362e..a8d4f7d 100644
--- a/config/checkstyle/checkstyle.xml
+++ b/config/checkstyle/checkstyle.xml
@@ -70,7 +70,11 @@ under the License.
             <property name="max" value="500"/>
             <property name="countEmpty" value="false"/>
         </module>
-        <module name="ParameterNumber"/>
+        <module name="ParameterNumber">
+            <property name="max" value="10"/>
+            <property name="tokens" value="METHOD_DEF"/>
+            <property name="ignoreOverriddenMethods" value="true"/>
+        </module>
 
         <!-- Checks for whitespace -->
         <module name="EmptyForIteratorPad"/>