-
Notifications
You must be signed in to change notification settings - Fork 18
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.
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
[WIP] add some missing ssb-gossip config (without README) #40
Conversation
@mixmix what about #39 ? |
Although, I can certainly see the advantage of having aggregate documentation for all configuration options. |
@@ -27,7 +27,11 @@ module.exports = function setDefaults (name, config) { | |||
hops: 3 | |||
}, | |||
gossip: { | |||
connections: 3 | |||
connections: 3, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO - remove connections, it seems to not be used anywhere!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we could make it work
Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward? |
This thread is still relevant
…On Mon, 13 May 2019, 17:58 stale[bot], ***@***.***> wrote:
Is this still relevant? If so, what is blocking it? Is there anything you
can do to help move it forward?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#40 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAUK3HVET5DKWZ56W5NZ4SDPVD7O3ANCNFSM4GWLWUKA>
.
|
Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward? |
These are more options that
ssb-config
reads ... I don't know what they are, they're not really documented in theapi.md
... I don't know if they should be in here.Putting them in the
defaults.js
is a start .. ?