dolittle-obsolete / cli

Command Line Interace for working with the Dolittle platform
MIT License
0 stars 2 forks source link

Add commandhandler command syntax change #79

Open TomasEkeli opened 5 years ago

TomasEkeli commented 5 years ago

Now the syntax is

dolittle add commandhandler {new_commandhandler_name}

which leads to a selction of which command to handle

I feel it should be

dolittle add commanhandler {existing_command}

which might ask what to call the command, I think it should just call it {existing_command}_handler

Even better would be to ommit "add" and add a new "handle" command:

dolittle handle command {existing_command}

┆Issue is synchronized with this Asana task

einari commented 5 years ago

The add is for consistency throughout.

I'm not sure I follow the first part. Are you talking about one command handler file per command? If so, that's not what we're seeing in all projects. One tends to group handlers for processes - or even aggregate roots.