Bug fix DL4001

Output a message only at the use location when using curl and wget in both directions

cmd/cli.go

@@ -1,3 +1,4 @@
+// Package command line tool to analyzer dockerfile.
 package cmd
 
 import (

linter/rules/dl4001.go

@@ -9,20 +9,25 @@ import (
 // validateDL4001 Either use Wget or Curl but not both
 func validateDL4001(node *parser.Node) (rst []ValidateResult, err error) {
 	isCurl, isWget := false, false
+	var numArr []int
 	for _, child := range node.Children {
 		switch child.Value {
 		case RUN:
 			for _, v := range strings.Fields(child.Next.Value) {
 				switch v {
 				case "curl":
 					isCurl = true
+					numArr = append(numArr, child.StartLine)
 				case "wget":
 					isWget = true
+					numArr = append(numArr, child.StartLine)
 				}
 			}
 		}
 		if isCurl && isWget {
-			rst = append(rst, ValidateResult{line: child.StartLine, addMsg: ""})
+			for _, num := range numArr {
+				rst = append(rst, ValidateResult{line: num, addMsg: ""})
+			}
 		}
 	}
 	return rst, nil

linter/rules/dl4001_test.go

@@ -16,6 +16,7 @@ RUN wget http://google.com
 RUN curl http://bing.com
 `,
 			expectedRst: []ValidateResult{
+				{line: 2, addMsg: ""},
 				{line: 3, addMsg: ""},
 			},
 			expectedErr: nil,

linter/rules/dl4003.go

@@ -4,7 +4,7 @@ import (
 	"github.com/moby/buildkit/frontend/dockerfile/parser"
 )
 
-// validateDL4003 Either use Wget or Curl but not both
+// validateDL4003 Multiple CMD instructions found.
 func validateDL4003(node *parser.Node) (rst []ValidateResult, err error) {
 	isCmd := false
 	for _, child := range node.Children {