✨ Add woodpecker as known CI (#4336)

* Add woodpecker as known CI Signed-off-by: 6543 <[email protected]> * Update probes/testsRunInCI/impl.go Signed-off-by: 6543 <[email protected]> --------- Signed-off-by: 6543 <[email protected]>
ossf · Sep 11, 2024 · 6d9b2e4 · 6d9b2e4
1 parent d6fd917
commit 6d9b2e4
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 3 deletions.
diff --git a/docs/checks.md b/docs/checks.md
@@ -149,7 +149,7 @@ number of vulnerabilities that find their way into a project.
 The check works by looking for a set of CI-system names in GitHub `CheckRuns`
 and `Statuses` among the recent commits (~30). A CI-system is considered
 well-known if its name contains any of the following: appveyor, buildkite,
-circleci, e2e, github-actions, jenkins, mergeable, test, travis-ci.
+circleci, e2e, github-actions, jenkins, mergeable, test, travis-ci, woodpecker.
 
 Note: A project that fulfills this criterion with other tools may still receive
 a low score on this test. There are many ways to implement CI testing, and it is

diff --git a/docs/checks/internal/checks.yaml b/docs/checks/internal/checks.yaml
@@ -242,7 +242,7 @@ checks:
       The check works by looking for a set of CI-system names in GitHub `CheckRuns`
       and `Statuses` among the recent commits (~30). A CI-system is considered
       well-known if its name contains any of the following: appveyor, buildkite,
-      circleci, e2e, github-actions, jenkins, mergeable, test, travis-ci.
+      circleci, e2e, github-actions, jenkins, mergeable, test, travis-ci, woodpecker.
 
       Note: A project that fulfills this criterion with other tools may still receive
       a low score on this test. There are many ways to implement CI testing, and it is

diff --git a/probes/testsRunInCI/impl.go b/probes/testsRunInCI/impl.go
@@ -165,7 +165,7 @@ func isTest(s string) bool {
 	for _, pattern := range []string{
 		"appveyor", "buildkite", "circleci", "e2e", "github-actions", "jenkins",
 		"mergeable", "packit-as-a-service", "semaphoreci", "test", "travis-ci",
-		"flutter-dashboard", "Cirrus CI", "azure-pipelines",
+		"flutter-dashboard", "Cirrus CI", "azure-pipelines", "ci/woodpecker",
 	} {
 		if strings.Contains(l, pattern) {
 			return true

diff --git a/probes/testsRunInCI/impl_test.go b/probes/testsRunInCI/impl_test.go
@@ -238,6 +238,13 @@ func Test_isTest(t *testing.T) {
 			},
 			want: false,
 		},
+		{
+			name: "woodpecker",
+			args: args{
+				s: "ci/woodpecker/pr/test-release",
+			},
+			want: true,
+		},
 	}
 	for _, tt := range tests {
 		tt := tt