Option to disable expose of all docker containers

[s7anley]

In case someone wants to register containers manually.

[s7anley]

@emilevauge I'm strangling with default value when configuration file is used. I didn't find the way how to configure viper. I tried viper.SetDefault, but this value is ignore once viper settings are loaded - configuration.go:304. Any ideas?

[emilevauge]

@s7anley thanks for your contribution!
We are in the process of replacing viper/cobra by our own lib, it should be a lot easier to manage these use cases. I suggest we wait this refactor before merging your PR :)
WDYT?

[s7anley]

Sure, not a problem.

[bbinet]

+1, it would be great if it could be merged before 1.0 release.

[s7anley]

@emilevauge any news regarding own configuration lib?

[emilevauge]

The PR is here #403, will be merged soon :)

[emilevauge]

Hi @s7anley, the PR #403 is merged ;)

[s7anley]

@emilevauge thanks for update, I will take a look during weekend.

[emilevauge]

Ping ?

[emilevauge]

Hi @s7anley , I'm planning to get this PR in the 1.1 releases. Do you think it's possible ?

[vdemeester]

Carrying here #514

[s7anley]

Oh, I'm really sorry guys. Hopefully, next time I will do better work.

2016-07-11 23:02 GMT+02:00 Vincent Demeester notifications@github.com:

Closed #358 #358.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#358 (comment), or mute
the thread
/~https://github.com/notifications/unsubscribe/AAnEg5PqL4RI9S55QqMa0IKFWBY9LEIiks5qUq9cgaJpZM4IZbw8
.

[vdemeester]

@s7anley don't worry about it, I'm just carrying it (and fixing conflicts), but it's still your work and your commit 😉. You did a good job 👍