Description
As discussed in mailing list, our community decide to enable a more meaningful checkstyle configuration, specifically, BookKeeper's checkstyle configuration.
Break down implement steps
1. Introduce BookKeeper's checkstyle configuration.
2. Enable this checkstyle configuration per package, for the first iteration, we want to enable it in
1). zookeeper-server
2). zookeeper-jute
3). zookeeper-prometheus-metrics
UPDATE 2019-08-27:
x. Turn on checkstyle configuration in project level while suppress on zookeeper-contrib.
Attachments
Issue Links
- is related to
-
ZOOKEEPER-3469 Checkstyle rules for ensure proper javadocs
- Open
-
ZOOKEEPER-3509 Revisit log format
- Resolved
- relates to
-
ZOOKEEPER-3507 Revisit name patterns
- Open
-
ZOOKEEPER-3508 Strategy on line break
- Open
- supercedes
-
ZOOKEEPER-3446 Enable UnusedImports and RedundantImport checkstyle rules
- Resolved
-
ZOOKEEPER-3434 [FileTabCharacter] clear up the all the checkstyle violations in the zookeeper-server module
- Resolved
1.
|
enfore checkstyle in the zookeeper-server module and clean the package:admin and client | Resolved | Ling Mao |
|
||||||||
2.
|
Introduce BookKeeper checkstyle configuration | Resolved | Zili Chen |
|
||||||||
3.
|
Enable BookKeeper checkstyle configuration on zookeeper-jute | Resolved | Zili Chen |
|
||||||||
4.
|
Enable BookKeeper checkstyle configuration on zookeeper-promethus | Resolved | Zili Chen |
|
||||||||
5.
|
Enable BookKeeper checkstyle configuration on zookeeper-server | Resolved | Zili Chen |
|
||||||||
6.
|
Turn on BookKeeper checkstyle configuration at project level | Resolved | Zili Chen |
|