Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Rename vaults list command to vaults ls #310

Open
tegefaulkes opened this issue Oct 16, 2024 · 2 comments
Open

Rename vaults list command to vaults ls #310

tegefaulkes opened this issue Oct 16, 2024 · 2 comments
Assignees
@aryanjassal
Labels
development Standard development

Comments

@tegefaulkes
Copy link
Contributor

Specification

@CMCDragonkai mentioned that since the secrets list has become secrets ls now, we should rename the vaults list command the same.

So the vaults list command should be renamed to vaults ls with list as an alias.

Additional context

Tasks

  1. Rename Vaults list to vaults ls.
  2. Include list as a command alias.
  3. Update tests to reflect this change.
@tegefaulkes tegefaulkes added the development Standard development label Oct 16, 2024
@linear Linear
Copy link

linear bot commented Oct 16, 2024

ENG-431 Rename `vaults list` command to `vaults ls`

@CMCDragonkai
Copy link
Member

In general, anything that is list could be ls.

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

@aryanjassal aryanjassal

Labels
development Standard development
Milestone
No milestone
Development

No branches or pull requests
3 participants
@CMCDragonkai @tegefaulkes @aryanjassal