Skip to content

Commit

Permalink
add args validation (#134)
Browse files Browse the repository at this point in the history
  • Loading branch information
JunNishimura committed Jun 20, 2023
1 parent 8483466 commit 46b1c28
Showing 1 changed file with 25 additions and 20 deletions.
45 changes: 25 additions & 20 deletions cmd/rm.go
Original file line number Diff line number Diff line change
@@ -1,40 +1,45 @@
/*
Copyright © 2023 NAME HERE <EMAIL ADDRESS>
*/
package cmd

import (
"fmt"
"path/filepath"
"strings"

"github.com/spf13/cobra"
)

var (
rFlag bool
)

// rmCmd represents the rm command
var rmCmd = &cobra.Command{
Use: "rm",
Short: "A brief description of your command",
Long: `A longer description that spans multiple lines and likely contains examples
and usage of using your command. For example:
Cobra is a CLI library for Go that empowers applications.
This application is a tool to generate the needed files
to quickly create a Cobra application.`,
Run: func(cmd *cobra.Command, args []string) {
fmt.Println("rm called")
Short: "remove file from the working tree and the index",
Long: "remove file from the working tree and the index",
PreRunE: func(cmd *cobra.Command, args []string) error {
if client.RootGoitPath == "" {
return ErrGoitNotInitialized
}
return nil
},
RunE: func(cmd *cobra.Command, args []string) error {
// args validation
for _, arg := range args {
// check if the arg is registered in the Index
cleanedArg := filepath.Clean(arg)
cleanedArg = strings.ReplaceAll(cleanedArg, `\`, "/")
client.Idx.GetEntry([]byte())

Check failure on line 34 in cmd/rm.go

View workflow job for this annotation

GitHub Actions / test (ubuntu-latest)

missing argument in conversion to []byte

Check failure on line 34 in cmd/rm.go

View workflow job for this annotation

GitHub Actions / lint

missing argument in conversion to []byte (typecheck)

Check failure on line 34 in cmd/rm.go

View workflow job for this annotation

GitHub Actions / test (macos-latest)

missing argument in conversion to []byte

Check failure on line 34 in cmd/rm.go

View workflow job for this annotation

GitHub Actions / test (windows-latest)

missing argument in conversion to []byte
}

return nil
},
}

func init() {
rootCmd.AddCommand(rmCmd)

// Here you will define your flags and configuration settings.

// Cobra supports Persistent Flags which will work for this command
// and all subcommands, e.g.:
// rmCmd.PersistentFlags().String("foo", "", "A help for foo")

// Cobra supports local flags which will only run when this command
// is called directly, e.g.:
// rmCmd.Flags().BoolP("toggle", "t", false, "Help message for toggle")
rmCmd.Flags().BoolVarP(&rFlag, "rec", "r", false, "allow recursive removal")
}

0 comments on commit 46b1c28

Please sign in to comment.