toolshed/abra
4
2
Fork 18
Code Issues 32 Pull Requests 3 Packages Releases 40 Activity

refactor: centralise app name validation
All checks were successful
continuous-integration/drone/push Build is passing
Details

Browse Source
This commit is contained in:
decentral1se
2021-09-05 22:04:48 +02:00
parent cc249e8187
commit ec40d88134
15 changed files with 41 additions and 82 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
cli/app/volume.go
View File

@ -2,7 +2,6 @@ package app
import (
"context"
"errors"
abraFormatter "coopcloud.tech/abra/cli/formatter"
"coopcloud.tech/abra/cli/internal"
@ -34,10 +33,7 @@ var appVolumeListCommand = &cli.Command{
Usage: "list volumes associated with an app",
Aliases: []string{"ls"},
Action: func(c *cli.Context) error {
appName := c.Args().First()
if appName == "" {
internal.ShowSubcommandHelpAndError(c, errors.New("no app name provided!"))
}
appName := internal.ValidateAppNameArg(c)
ctx := context.Background()
server := getAppServer(appName)
@ -68,10 +64,7 @@ var appVolumeRemoveCommand = &cli.Command{
internal.ForceFlag,
},
Action: func(c *cli.Context) error {
appName := c.Args().First()
if appName == "" {
internal.ShowSubcommandHelpAndError(c, errors.New("no app name provided!"))
}
appName := internal.ValidateAppNameArg(c)
server := getAppServer(appName)
ctx := context.Background()
volumeList, err := client.GetVolumes(ctx, server, appName)