search

apache / distributedlog

Apache DistributedLog
Apache License 2.0
185 stars 74 forks source link

Enable checkstyle for all modules #165

Closed sijie closed 7 years ago

sijie commented 7 years ago

FEATURE REQUEST

  1. Please describe the feature you are requesting.

checkstyle is disabled in some modules. it would be good to enable checkstyle on all modules, to enforce code styles.

  1. Indicate the importance of this issue to you (blocker, must-have, should-have, nice-to-have). Are you currently using any workarounds to address this issue?

nice-to-have

  1. Provide any additional detail on your proposed use case for this feature.

we can follow apache/bookkeeper#230 to divide this into multiple small tasks

ArvinDevel commented 7 years ago

checkstyle has been applied to several modules, including distributedlog-proxy-server,distributedlog-proxy-client, distributedlog-protocol,distributedlog-core-twitter,distributedlog-common and distributedlog-benchmark. So add checkstyle to below modules: