Making delete more elegant

Author: italolelis

cmd/repo_delete.go

@@ -13,9 +13,7 @@ import (
 
 type (
 	// DeleteRepoOpts are the flags for the delete repo command
-	DeleteRepoOpts struct {
-		Name string
-	}
+	DeleteRepoOpts struct{}
 )
 
 // NewDeleteRepoCmd creates a new delete repo command
@@ -25,17 +23,16 @@ func NewDeleteRepoCmd(ctx context.Context) *cobra.Command {
 		Use:   "delete",
 		Short: "Deletes a github repository",
 		RunE: func(cmd *cobra.Command, args []string) error {
-			return RunDeleteRepo(ctx, opts)
+			return RunDeleteRepo(ctx, args[0], opts)
 		},
+		Args: cobra.MinimumNArgs(1),
 	}
 
-	cmd.Flags().StringVarP(&opts.Name, "name", "n", "", "The name of the repository")
-
 	return cmd
 }
 
 // RunDeleteRepo runs the command to delete a repository
-func RunDeleteRepo(ctx context.Context, opts *DeleteRepoOpts) error {
+func RunDeleteRepo(ctx context.Context, name string, opts *DeleteRepoOpts) error {
 	logger := log.WithContext(ctx)
 	cfg := config.WithContext(ctx)
 	githubClient := gh.WithContext(ctx)
@@ -48,12 +45,12 @@ func RunDeleteRepo(ctx context.Context, opts *DeleteRepoOpts) error {
 		return errors.New("please provide an organization")
 	}
 
-	_, err := githubClient.Repositories.Delete(ctx, org, opts.Name)
+	_, err := githubClient.Repositories.Delete(ctx, org, name)
 	if err != nil {
 		return errwrap.Wrapf("Could not delete repository: {{err}}", err)
 	}
 
-	logger.Info("Repository %s deleted!", opts.Name)
+	logger.Infof("Repository %s deleted!", name)
 
 	return nil
 }