Ignore excluded entries when unpinning (#17)

sethvargo · May 24, 2022 · b2eee82 · b2eee82
1 parent 3502a40
commit b2eee82
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/parser/parser.go b/parser/parser.go
@@ -167,7 +167,7 @@ func Unpin(m *yaml.Node) error {
 		return nil
 	}
 
-	if m.LineComment != "" {
+	if m.LineComment != "" && !shouldExclude(m.LineComment) {
 		if v, rest := extractOriginalFromComment(m.LineComment); v != "" {
 			m.Value = v
 			m.LineComment = rest

diff --git a/parser/parser_test.go b/parser/parser_test.go
@@ -222,6 +222,11 @@ func TestUnpin(t *testing.T) {
 - uses: "i/am@pinned" # comment
 `,
 		},
+		{
+			name: "exclude_comment",
+			in:   `uses: "my/repo@v0" # ratchet:exclude more comment`,
+			exp:  `uses: "my/repo@v0" # ratchet:exclude more comment`,
+		},
 	}
 
 	for _, tc := range cases {