You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@bookkeeper.apache.org by GitBox <gi...@apache.org> on 2017/12/06 16:27:06 UTC

[GitHub] acoburn opened a new pull request #819: ISSUE #230: Enable checkstyle on some test classes

acoburn opened a new pull request #819: ISSUE #230: Enable checkstyle on some test classes
URL: https://github.com/apache/bookkeeper/pull/819
 
 
   Part of #230, this enables checkstyle verification on some
   of the test code. This does not do checkstyle validation on
   the client or bookie package test code.
   
   Enabling checkstyle on the entire test codebase would result in a _very_ large PR. This is an attempt to break it up into smaller pieces. The changes here are almost exclusively related to whitespace, javadocs and import order.
   
   One public change of note is in `replication.AuditorLedgerCheckerTest`: the `_testDelayedAuditOfLostBookies` method is changed to `testInnerDelayedAuditOfLostBookies`.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services