-
Type:
Improvement
-
Status: Closed
-
Priority:
Major
-
Resolution: Implemented
-
Affects Version/s: None
-
Fix Version/s: None
-
Component/s: None
-
Labels:None
-
Sprint:OFBiz Community Day (Nov 2020)
Currently Checkstyle returns 222 occurrences of the message "Utility classes should not have a public or default constructor."
This is due to https://checkstyle.sourceforge.io/config_design.html#HideUtilityClassConstructor
I don't think we need to follow the "prevents calls from subclass" suggestion there.
I rather suggest to comment out the related class in checkstyle.xml:
ie <!-- <module name="HideUtilityClassConstructor"/> -->
Then should remain only 280 errors and hopefully most will be easy to fix. At least they will be easier to spot...