Hangleton commented on code in PR #13558:
URL: https://github.com/apache/kafka/pull/13558#discussion_r1205669500
##########
checkstyle/suppressions.xml:
##########
@@ -41,6 +41,8 @@
<suppress checks="NPathComplexity"
files="(ClusterTestExtensions|KafkaApisBuilder).java"/>
<suppress checks="MethodLength"
files="(KafkaClusterTestKit).java"/>
+ <suppress checks="ImportControl"
+
files="(ZkBrokerRegistrationTest|DelegatingRequest|InstrumentedRequestProcessor|InstrumentedZooKeeperServer|ReceiptEvent|ZkBrokerRegistrationStubs).java"/>
Review Comment:
Added the packages to import-control-core in [this
commit](https://github.com/apache/kafka/pull/13558/commits/03d444b94de2494548b61cd0458c842c05872e3f).
I had to kept a suppression rule for the package `org.apache.zookeeper`
because it is not part of any import control and Checkstyle would not accept it
without a proper import control definition.
--
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]
