Skip to content
This repository has been archived by the owner on Jan 9, 2024. It is now read-only.

Fix #217 #218

Merged
merged 2 commits into from
Mar 3, 2018
Merged

Fix #217 #218

merged 2 commits into from
Mar 3, 2018

Conversation

AngusP
Copy link
Contributor

@AngusP AngusP commented Sep 13, 2017

Adds cluster-bus-port to each node's dict for both >=4.0 nodes and <4.0 to keep consistency

…ll back to client port + 10000 if the @ is not present
@Grokzen
Copy link
Owner

Grokzen commented Nov 1, 2017

@AngusP please add a line in the changelog and i will merge this one.

@Grokzen
Copy link
Owner

Grokzen commented Nov 1, 2017

Linking this to #217

@AngusP
Copy link
Contributor Author

AngusP commented Nov 28, 2017

@Grokzen sorry, just seen this, will do

@coveralls
Copy link

coveralls commented Nov 28, 2017

Coverage Status

Coverage increased (+1.7%) to 86.118% when pulling 9df2266 on AngusP:unstable into 078867b on Grokzen:unstable.

@AngusP
Copy link
Contributor Author

AngusP commented Dec 16, 2017

@Grokzen that's all sorted, if you don't need anything else done then I think this'd be OK to merge 😄

@Grokzen Grokzen merged commit 0d5fac5 into Grokzen:unstable Mar 3, 2018
@Grokzen
Copy link
Owner

Grokzen commented Mar 3, 2018

Sorry for taking so long. Thanks for the contribution @AngusP :]

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants