reswqa commented on code in PR #22619:
URL: https://github.com/apache/flink/pull/22619#discussion_r1203657255
##########
tools/maven/checkstyle.xml:
##########
@@ -180,6 +180,11 @@ This file is based on the checkstyle file of Apache Beam.
<property name="severity" value="error"/>
</module>
+ <module name="IllegalImport">
+ <property name="illegalPkgs"
+ value="org.mockito, org.powermock"/>
+ <message key="import.illegal" value="{0}. Please refer
to the coding guidelines."/>
Review Comment:
It can be explained more clearly here like `we should avoid using mocks,
please refer to the coding
guidelines[https://flink.apache.org/how-to-contribute/code-style-and-quality-common/#avoid-mockito---use-reusable-test-implementations]`
or other better descriptions.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
