Code review comment for lp://staging/~mathieu-lonjaret/gozk/zookeeper

Revision history for this message
Gustavo Niemeyer (niemeyer) wrote :

LGTM

Just one detail:

https://codereview.appspot.com/5445048/diff/1/zk_test.go
File zk_test.go (right):

https://codereview.appspot.com/5445048/diff/1/zk_test.go#newcode1
zk_test.go:1: package zk_test
The package name in all tests should be renamed to "zookeeper_test" as
well.

https://codereview.appspot.com/5445048/

« Back to merge proposal