peterldowns / localias

custom local domain aliases for local dev servers

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

suggestion to change command layout for alias related commands

gedw99 opened this issue · comments

commented

Suggest that the cli would clearer if it was like this:

localias alias ls
localias alias clear
localias alias set <alias> <port>
localias alias delete <alias>

All the others are nicely done.

This is subjective, but I don't think it makes sense to hide the most common logic behind a subcommand — the purpose of the localias binary is to manage aliases, so those commands should be at the root. Do you see how with your suggestion you'd have to type the word alias twice in a row? localias alias.

I'm going to close this as wontfix, but I will consider this when making future changes to the CLI. I appreciate the suggestion. If enough people are interested I would be willing to reconsider this decision.