MatrixAI / Polykey-CLI

Polykey CLI - Open Source Decentralized Secret Sharing System for Zero Trust Workflows
https://polykey.com
GNU General Public License v3.0
6 stars 3 forks source link

Rename `vaults list` command to `vaults ls` #310

Closed tegefaulkes closed 6 days ago

tegefaulkes commented 2 weeks ago

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.
linear[bot] commented 2 weeks ago

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

CMCDragonkai commented 2 weeks ago

In general, anything that is list could be ls.

CMCDragonkai commented 1 week ago

You want to have list|ls available for these sorts of commands.

CMCDragonkai commented 4 days ago

Short versions of all commands should exist like remove|rm if it makes sense...

aryanjassal commented 3 days ago

Short versions of all commands should exist like remove|rm if it makes sense...

Oh. I'll make a new issue to convert all commands, vault or otherwise, to have short/long versions where applicable.

aryanjassal commented 3 days ago

Polykey-CLI#323