-
Notifications
You must be signed in to change notification settings - Fork 140
Deprecate in favour of docker/engine-api #238
Comments
I would say event management, it's pretty rough in https://github.com/docker/engine-api |
@vieux Specifically what about event management is lacking? I want to open an issue against docker/engine-api about this, but not sure exactly what to say. :) |
@bfirsh all the complicated code here https://github.com/samalba/dockerclient/blob/master/dockerclient.go#L472 that parses events and return them via channel, etc... last time I looked the only thing in engine-api was a |
/cc @vdemeester (we were talking about this a while back, re: events) |
@bfirsh @vieux yeah, the |
@vieux @vdemeester Thanks, this is really helpful. @vdemeester Agreed something like that should probably be in engine-api. I've opened docker/engine-api#246 to keep track. |
A bunch of projects use this library, so we probably shouldn't abandon it, but it would perhaps be sensible to suggest people try using https://github.com/docker/engine-api instead.
Is there anything missing from docker/engine-api that would stop users from being able to use it instead of this library?
/cc @samalba @vieux
The text was updated successfully, but these errors were encountered: