Add more command shorthands #307

Merged
6543 merged 2 commits from noerw/tea:aliases into master 4 months ago
noerw commented 4 months ago

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 4 months ago
noerw added the
kind/breaking
kind/enhancement
labels 4 months ago
noerw added 1 commit 4 months ago
2cac4f9678
add more command aliases
6543 reviewed 4 months ago
cmd/releases.go
var CmdReleases = cli.Command{
Name: "releases",
Aliases: []string{"release"},
Aliases: []string{"release", "r"},
6543 commented 4 months ago
Poster
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 4 months ago
Poster

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 4 months ago
6543 left a comment

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

@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
@lunny https://github.com/go-gitea/gitea/issues/13923
lunny approved these changes 4 months ago
6543 added 1 commit 4 months ago
753d9b3c9e Merge branch 'master' into aliases
6543 merged commit 782a6318f3 into master 4 months ago
6543 referenced this issue from a commit 4 months ago
noerw deleted branch aliases 4 months ago

Reviewers

6543 approved these changes 4 months ago
lunny approved these changes 4 months 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.