pingo icon indicating copy to clipboard operation
pingo copied to clipboard

Abandon additional transport method parameter

Open napsy opened this issue 9 years ago • 3 comments

Most of the time, users won't care what is the underlaying communication protocol. So, I propose:

func NewPlugin(path string, params ...string) *Plugin

To change the default communication protocol, expose:

func SetProto(proto string)

The API becomes clearer. E.g.:

p := pingo.NewPlugin("plugins/hello-world/hello-world")

napsy avatar Apr 28 '15 07:04 napsy

Agreed. This would also help implementing automatic switching of protocol.

dullgiulio avatar Apr 28 '15 07:04 dullgiulio

Would be nice to use RPC over stdin/stdout too.

kdar avatar May 01 '15 02:05 kdar

@kdar: I'll make a new issue for this.

dullgiulio avatar May 01 '15 20:05 dullgiulio