lhotari commented on PR #2: URL: https://github.com/apache/pulsar-java-contrib/pull/2#issuecomment-2306507590
> Indeed, normally it is enough to just use spotless when creating a new warehouse. But as a Pulsar incubation project. Because some of the functions may be contributed back to pulsar in the future. So I still added checkstyle and kept it consistent with pulsar. Of course, I could be wrong. yes you are right. We can keep the co-existence of checkstyle. Spotless references Avro as an example and they also use checkstyle and spotless together: https://github.com/apache/avro/blob/8026c8ffe4ef67ab419dba73910636bf2c1a691c/lang/java/pom.xml#L285-L332 -- 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]
