Skip to content

Alias list as ls for all commands #5214

Merged
mislav merged 2 commits intocli:trunkfrom
CGA1123:cga1123/gh-cs-ls
Feb 17, 2022
Merged

Alias list as ls for all commands #5214
mislav merged 2 commits intocli:trunkfrom
CGA1123:cga1123/gh-cs-ls

Conversation

@CGA1123
Copy link
Contributor

@CGA1123 CGA1123 commented Feb 16, 2022

I always get tripped up whenever trying to list my Codespaces, adding
ls as an alias for list feels natural enough!

Closes: #2602

@CGA1123 CGA1123 requested a review from a team as a code owner February 16, 2022 13:13
@cliAutomation
Copy link
Collaborator

Hi! Thanks for the pull request. Please ensure that this change is linked to an issue by mentioning an issue number in the description of the pull request. If this pull request would close the issue, please put the word 'Fixes' before the issue number somewhere in the pull request body. If this is a tiny change like fixing a typo, feel free to ignore this message.

@cliAutomation cliAutomation added the external pull request originating outside of the CLI core team label Feb 16, 2022
Copy link
Contributor

@josebalius josebalius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like it!

Copy link
Contributor

@veverkap veverkap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will save me so much time!

Copy link
Contributor

@mislav mislav left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Thank you for making the addition.

Would you also add an identical alias to other list commands in the codebase? There is a few that could also benefit from the alias. Ref. #2602

I always get tripped up whenever trying to list my codespaces, adding
`ls` as an alias to `list` feels natural enough.
@CGA1123 CGA1123 requested a review from a team as a code owner February 16, 2022 17:03
@CGA1123 CGA1123 requested review from vilmibm and removed request for a team February 16, 2022 17:03
@CGA1123 CGA1123 changed the title Alias gh cs list as gh cs ls Alias list as ls for all commands Feb 16, 2022
@CGA1123
Copy link
Contributor Author

CGA1123 commented Feb 16, 2022

Thanks for calling that out @mislav -- I think I have covered all of the list commands now.

I hadn't realised that there was an issue opened for this! My searching stills need to be improved -- thanks for linking to it. 😄

Co-authored-by: Mislav Marohnić <hi@mislav.net>
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i want to get my origin url

Copy link
Contributor

@mislav mislav left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

external pull request originating outside of the CLI core team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support ls as alias for "list" - subcommand

5 participants