[GitHub] incubator-rocketmq issue #45: [ROCKETMQ-54] Polish unit tests for rocketmq-n...

2017-01-19 Thread iskl
Github user iskl commented on the issue: https://github.com/apache/incubator-rocketmq/pull/45 I've merged this branch from remote master. The integration test on OSX are all passed. Thanks~ --- If your project is set up for it, you can reply to this email and have your

[GitHub] incubator-rocketmq issue #45: [ROCKETMQ-54] Polish unit tests for rocketmq-n...

2017-01-19 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/incubator-rocketmq/pull/45 [![Coverage Status](https://coveralls.io/builds/9760902/badge)](https://coveralls.io/builds/9760902) Coverage increased (+12.3%) to 25.826% when pulling

[GitHub] incubator-rocketmq issue #45: [ROCKETMQ-54] Polish unit tests for rocketmq-n...

2017-01-18 Thread stevenschew
Github user stevenschew commented on the issue: https://github.com/apache/incubator-rocketmq/pull/45 In the OSX system, the integration test does not pass. We have already dealt with it. Please merge master fetch first and push again. @iskl --- If your project is set up for it, you

[GitHub] incubator-rocketmq issue #45: [ROCKETMQ-54] Polish unit tests for rocketmq-n...

2017-01-18 Thread stevenschew
Github user stevenschew commented on the issue: https://github.com/apache/incubator-rocketmq/pull/45 Review Ok --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes

[GitHub] incubator-rocketmq issue #45: [ROCKETMQ-54] Polish unit tests for rocketmq-n...

2017-01-17 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/incubator-rocketmq/pull/45 [![Coverage Status](https://coveralls.io/builds/9708169/badge)](https://coveralls.io/builds/9708169) Coverage increased (+1.1%) to 14.629% when pulling

[GitHub] incubator-rocketmq issue #45: [ROCKETMQ-54] Polish unit tests for rocketmq-n...

2017-01-17 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/incubator-rocketmq/pull/45 [![Coverage Status](https://coveralls.io/builds/9708169/badge)](https://coveralls.io/builds/9708169) Coverage increased (+1.1%) to 14.629% when pulling