Merge pull request #52 from LinuxSuRen/fea/confirm-del-job

Confirm before user delete a job

app/cmd/common.go

@@ -5,6 +5,7 @@ import (
 	"fmt"
 
 	"github.com/AlecAivazis/survey"
+	"github.com/spf13/cobra"
 	"gopkg.in/yaml.v2"
 )
 
@@ -65,6 +66,10 @@ func (b *BatchOption) Confirm(message string) bool {
 	return true
 }
 
+func (b *BatchOption) SetFlag(cmd *cobra.Command) {
+	cmd.Flags().BoolVarP(&b.Batch, "batch", "b", false, "Batch mode, no need confirm")
+}
+
 // WatchOption for the resources which can be watched
 type WatchOption struct {
 	Watch    bool

app/cmd/job_delete.go

 package cmd
 
 import (
+	"fmt"
 	"log"
 
 	"github.com/linuxsuren/jenkins-cli/client"
@@ -8,12 +9,15 @@ import (
 )
 
 type JobDeleteOption struct {
+	BatchOption
 }
 
 var jobDeleteOption JobDeleteOption
 
 func init() {
 	jobCmd.AddCommand(jobDeleteCmd)
+	jobDeleteOption.SetFlag(jobDeleteCmd)
+	// jobDeleteCmd.Flags().BoolVarP(&jobDeleteOption.Batch, "batch", "b", false, "Batch mode, no need confirm")
 }
 
 var jobDeleteCmd = &cobra.Command{
@@ -27,6 +31,9 @@ var jobDeleteCmd = &cobra.Command{
 		}
 
 		jobName := args[0]
+		if !jobDeleteOption.Confirm(fmt.Sprintf("Are you sure to delete job %s ?", jobName)) {
+			return
+		}
 
 		jenkins := getCurrentJenkins()
 		jclient := &client.JobClient{}