Add more command shorthands #307

Merged
6543 merged 2 commits from noerw/tea:aliases into master 2 years ago
noerw commented 2 years ago
Collaborator

fixes #305

One change is breaking: tea notif -> tea n

fixes #305 One change is breaking: `tea notif` -> `tea n`
noerw added this to the v0.7.0 milestone 2 years ago
noerw added the
kind/breaking
kind/enhancement
labels 2 years ago
noerw added 1 commit 2 years ago
2cac4f9678
add more command aliases
6543 reviewed 2 years ago
Dismissed
var CmdReleases = cli.Command{
Name: "releases",
Aliases: []string{"release"},
Aliases: []string{"release", "r"},
6543 commented 2 years ago
Collaborator

what about repo shortcut?

i think rel & rep are the shortest unique here ...

what about repo shortcut? i think `rel` & `rep` are the shortest unique here ...
noerw commented 2 years ago
Poster
Collaborator

True. though rep is not really shorter than repo, so i went the route of shortining the longer word.. idk

True. though `rep` is not really shorter than `repo`, so i went the route of shortining the longer word.. idk
6543 marked this conversation as resolved
6543 approved these changes 2 years ago
Dismissed
6543 left a comment
Collaborator

Webshould look if we can make some sort of "Longest Prefix Matching" if urfave/cli supports it

(As son as it is unique and match e.g. rep for repo it gets valide)

Webshould look if we can make some sort of "Longest Prefix Matching" if urfave/cli supports it (As son as it is unique and match e.g. rep for repo it gets valide)
Owner

Sorry, I may express myself wrongly. I mean users could customize their alias. Just like git alias command, we can have tea alias or change that on tea config file.

Sorry, I may express myself wrongly. I mean users could customize their alias. Just like `git alias` command, we can have `tea alias` or change that on tea config file.
Poster
Collaborator

@lunny Ah ok. oof, thats not gonna be simple. Less time will be spent with telling users to build from source & changing their preferred aliases ;)

@6453 agree that would be great. but i don't see that coming in urfave/cli. I requested it anyways :D

@lunny Ah ok. oof, thats not gonna be simple. Less time will be spent with telling users to build from source & changing their preferred aliases ;) @6453 agree that would be great. but i don't see that coming in urfave/cli. I [requested it anyways](https://github.com/urfave/cli/issues/1215) :D
Owner

Hm, why the labels comments missed. :(

Hm, why the labels comments missed. :(
Poster
Collaborator
@lunny https://github.com/go-gitea/gitea/issues/13923
lunny approved these changes 2 years ago
Dismissed
6543 added 1 commit 2 years ago
753d9b3c9e Merge branch 'master' into aliases
6543 merged commit 782a6318f3 into master 2 years ago
6543 referenced this issue from a commit 2 years ago
noerw deleted branch aliases 2 years ago
continuous-integration/drone/pr Build is passing
The pull request has been merged as 782a6318f3.
Sign in to join this conversation.
Loading…
There is no content yet.