Sort host list by name

[dilijev]

No description provided.

[bterlson]

What's wrong with always sorting? First-to-complete gets you results earlier potentially, but it feels like overkill to require an option for it...

[dilijev]

@bterlson Always sorting makes some kind of sense, but the default listing is in config-order which might be significant to the user. Didn't want to make that call necessarily.

By the way this isn't for running scripts. Those are now in alpha-sorted order by default when using --coalesce (see #29). This is for the --list option to display configured hosts.

[bterlson]

I think sorting always is fine, personally. I see it as the moral equivalence of git branch sorting branch names (rather than doing creation order or something). @rwaldron any preferences?

[rwaldron]

When I reviewed this last week I thought a bit about the sorting and wondered whether a user might also want to sort by other columns (by providing the column name) and I spent sometime sketching that out; but ultimately I decided against pursuing it and reasoned that if eshost-cli wanted to grow that capability, then it could without an issue and that this was a reasonable place to start, while also not that could be extended later.

But then...

I think sorting always is fine, personally. I see it as the moral equivalence of git branch sorting branch names

...is definitely compelling. So +1 to always sorting.

[dilijev]

Okay, always sorting is vastly simpler. I doubt the small audience of this tool is going to have someone whose workflow is broken by things being in a different order. I'll make the update soon.

[dilijev]

@bterlson lol one line change when it's not optional