peterldowns / localias

custom local domain aliases for local dev servers
MIT License
583 stars 3 forks source link

suggestion to change command layout for alias related commands #12

Closed gedw99 closed 1 year ago

gedw99 commented 1 year ago

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.

peterldowns commented 1 year ago

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.