fixup! ci: add prcontext

coder · ammario · Jul 24, 2022 · Jul 23, 2022 · Jul 23, 2022 · Jul 23, 2022
commit f2de14a9bb0b314daeb10b83001ac904e13fbc3a
diff --git a/.github/workflows/prcontext/main.go b/.github/workflows/prcontext/main.go
@@ -32,8 +32,8 @@ func main() {
 		return
 	}
 
-	_, _ = fmt.Printf("---\n%s\n---\n", c.Event.PullRequest.Body)
+	_, _ = fmt.Printf("::group::{PR Body}\n%s\n::endgroup::\n", c.Event.PullRequest.Body)
 
 	skips := parseBody(c.Event.PullRequest.Body)
-	_, _ = fmt.Printf("::set-output name=skips::[%s]\n", strings.Join(skips, " "))
+	_, _ = fmt.Printf("::echo::on\n::set-output name=skips::[%s]\n", strings.Join(skips, " "))
 }
diff --git a/.github/workflows/prcontext/parse.go b/.github/workflows/prcontext/parse.go
@@ -3,8 +3,6 @@ package main
 import (
 	"regexp"
 	"strings"
-
-	"github.com/coder/flog"
 )
 
 const ciSkipPrefix = "ci-skip"
@@ -19,7 +17,7 @@ func parseBody(body string) (skips []string) {
 	for i := range matches {
 		for j := range matches[i] {
 			v := matches[i][j]
-			flog.Info("%q", v)
+			// flog.Info("%q", v)
 			if !strings.Contains(v, ciSkipPrefix) {
 				skipMatches = append(skipMatches, strings.Split(v, " ")...)
 			}

diff --git a/.github/workflows/prcontext/parse_test.go b/.github/workflows/prcontext/parse_test.go
@@ -25,7 +25,7 @@ func Test_parseBody(t *testing.T) {
 		{"no directive", args{"test pr 123\n\n"}, nil},
 		{"single dir single skip", args{"test pr [ci-skip dog] 123\n\n"}, []string{"dog"}},
 		{"double dir double skip", args{"test pr [ci-skip dog] [ci-skip cat] 123\n\n"}, []string{"dog", "cat"}},
-		{"single dir double skip", args{"test pr [ci-skip dog cat] 123\n\n"}, []string{"dog", "cat"}},
+		{"single dir double skip", args{"test pr [ci-skip test/go/postgres cat] 123\n\n"}, []string{"test/go/postgres", "cat"}},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {