search

mcollina / hyperemitter

Horizontally Scalable EventEmitter powered by a Merkle DAG
ISC License
70 stars 2 forks source link

CLI protobuf schema should have a flag --schema #4

Closed mcdonnelldean closed 9 years ago

mcdonnelldean commented 9 years ago

Consider:

hypem --db db --port 1234 fixture/messages.proto
hypem --db db2 --target-port 1234 fixture/messages.proto

It is hard to tell what fixute/messages.proto is for, a simple flag would add clarity

mcollina commented 9 years ago

true, it was there but then I removed it somehow. I don't mind if you want to add it back :D.

mcdonnelldean commented 9 years ago

Awesome, I'll do this so, are you happy with --schema?

mcollina commented 9 years ago

You can support both :) Il giorno mer 6 mag 2015 alle 11:13 Dean McDonnell notifications@github.com ha scritto:

Awesome, I'll do this so, are you happy with --schema?

— Reply to this email directly or view it on GitHub https://github.com/mcollina/hyperemitter/issues/4#issuecomment-99392889.

mcdonnelldean commented 9 years ago

Even better idea!

mcdonnelldean commented 9 years ago

I'll do this one before the weekend.