search

kr / hk

Fast Heroku client
https://hk.heroku.com/
77 stars 6 forks source link

break out ls command #71

Closed bgentry closed 10 years ago

bgentry commented 10 years ago

Back around 5592ea093930a671535a50e90aa43e91ed4d9e80, multiple commands for listing things were combined into a single command, ls (apps, dynos, releases, and addons).

@kr has mentioned that this was a mistake, so we should split these out into their own commands.

In keeping with the nouns used in the new v3 API, I propose:

$ hk apps
$ hk dynos
$ hk releases
$ hk addons

Thoughts?

bgentry commented 10 years ago

/cc @freeformz

kr commented 10 years ago

This sounds fine. My only comment is a general design strategy: if there exists a strong unix precedent, fitting in with unix is usually more important than fitting in with heroku. For example, it's hk env, not hk config.

But we should also try to Say What We Mean, which means a hypothetical hk ps command ought to really list processes, not just dynos.

The specific commands listed above look great to me.

Do you have thoughts on how they'll fit into the larger picture of all hk commands? It might be worthwhile getting together over some post-it notes or a whiteboard for this.