New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix makeBalancedSubscriptions #632

Merged

icebob merged 4 commits into moleculerjs:next from AndreMaz:next

Dec 17, 2019

Member

AndreMaz commented Dec 16, 2019

📝 Description

🎯 Relevant issues

💎 Type of change

Bug fix (non-breaking change which fixes an issue)

AndreMaz added 3 commits

December 16, 2019 21:50


          Fix makeBalancedSubscriptions

92ef796


          Fix test


          Merge remote-tracking branch 'upstream/next' into next

f3d3ea2

coveralls commented Dec 16, 2019

Pull Request Test Coverage Report for Build 2165

1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 94.268%

Files with Coverage Reduction	New Missed Lines	%
src/transporters/fake.js	2	91.3%

Totals
Change from base Build 2164:	-0.02%
Covered Lines:	6585
Relevant Lines:	6751

💛 - Coveralls

coveralls commented Dec 16, 2019 •

edited

Pull Request Test Coverage Report for Build 2166

1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 94.253%

Files with Coverage Reduction	New Missed Lines	%
src/transporters/fake.js	2	91.3%

Totals
Change from base Build 2164:	-0.03%
Covered Lines:	13169
Relevant Lines:	13502

💛 - Coveralls


          Fix tests

2b672ae

icebob approved these changes

View reviewed changes

Member

icebob left a comment

Good job, thanks!

icebob merged commit a149f04 into moleculerjs:next

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment