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.

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

Add some shorthand aliases for commands+subcommands #376

Merged
merged 8 commits into from
Mar 2, 2022

Conversation

e-zk
Copy link
Contributor

@e-zk e-zk commented Mar 2, 2022

This adds a few shorthand aliases for commands+subcommands.

  • read the CONTRIBUTING guidelines
  • raised a GitHub issue or discussed it on the projects chat beforehand
  • added unit tests
  • added integration tests
  • updated documentation if needed
  • updated CHANGELOG.md

Ref: #361.

New command aliases summary:

  • apikeys: apikey, api
  • generate: gen
  • namespaces: namespace, ns, user, users
  • nodes: node, machines, machine
  • preauthkeys: preauthkey, authkey, pre
  • routes: route, r

Subcommand aliases summary:

  • list: ls, show
  • create: c, new
  • rename: mv
  • delete: del, rm
  • expire: logout, exp, e

Suggestions for additions+changes welcome.

Also, a question: am I meant to run make gen and push the changed files in gen/?

e-zk added 6 commits March 2, 2022 14:27
- `r` is alias for `route` command
- `ls`, or `show` is alias for `list` subcommand
- `node`, `machine`, `machines` are aliases for `nodes` command
- `ls`, `show` aliases for `list` subcommand
- `logout`, `exp`, `e` are aliases for `expire` subcommand
- `del` is an alias for `delete` subcommand
- `namespace`, `ns`, `user`, `users` are aliases for `namespaces`
   command
- `c`, `new` are aliases for the `create` subcommand
- `delete` is an alias for the `destroy` subcommand
- `mv` is an alias for the `rename` subcommand
- `ls`, `show` are aliases for the `list` subcommand
- `apikey`, `api` are aliases for `apikeys` command
- `ls`, `show` are aliases for `list` subcommand
- `c`, `new` are aliases for `create` subcommand
- `revoke`, `exp`, `e` are aliases for the `expire` subcommand
- `preauthkey`, `authkey`, `pre` are aliases for `preauthkey` command
- `ls`, `show` are aliases for `list` subcommand
- `c`, `new` are aliases for `create` subcommand
- `revoke`, `exp`, `e` are aliases for `expire` subcommand
@e-zk e-zk requested review from juanfont and kradalby as code owners March 2, 2022 06:05
kradalby
kradalby previously approved these changes Mar 2, 2022
Copy link
Collaborator

@kradalby kradalby 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, thanks :)

@kradalby
Copy link
Collaborator

kradalby commented Mar 2, 2022

@e-zk I was a bit quick, would you mind adding a note to the changelog?

@e-zk
Copy link
Contributor Author

e-zk commented Mar 2, 2022

@kradalby sure thing.

Preferably, this'd fall under "Features", or "Changes"?

@e-zk
Copy link
Contributor Author

e-zk commented Mar 2, 2022

Added a note to changelog under "Features"

@kradalby kradalby merged commit 63d8711 into juanfont:main Mar 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants