[GitHub] zookeeper issue #550: ZOOKEEPER-3022 - Iteration 1 - docs, it

2018-06-27 Thread nkalmar
Github user nkalmar commented on the issue: https://github.com/apache/zookeeper/pull/550 You're right @anmolnar , I also realised after seeing the PR that I should have added some text indicating this is part of the migration. I will add it in the future, thanks. ---

[GitHub] zookeeper issue #550: ZOOKEEPER-3022 - Iteration 1 - docs, it

2018-06-27 Thread anmolnar
Github user anmolnar commented on the issue: https://github.com/apache/zookeeper/pull/550 Thanks @nkalmar, looks good to me. One nitpick only: add "MAVEN MIGRATION" to the title of all these PRs in capitals, just to make it obvious for everybody that you're making progress. ---