Fixes

- Fix nil exception in packaging on https://github.com/OSGeo/gdal
- Add jenkins ci in ci tests, tested on https://github.com/jenkinsci/jenkins
- Generalize function name in code review check.

checks/ci_tests.go

@@ -113,7 +113,7 @@ func isTest(s string) bool {
 	l := strings.ToLower(s)
 
 	// Add more patterns here!
-	for _, pattern := range []string{"appveyor", "buildkite", "circleci", "e2e", "github-actions", "mergeable", "test", "travis-ci"} {
+	for _, pattern := range []string{"appveyor", "buildkite", "circleci", "e2e", "github-actions", "jenkins", "mergeable", "test", "travis-ci"} {
 		if strings.Contains(l, pattern) {
 			return true
 		}

checks/code_review.go

@@ -35,7 +35,7 @@ func DoesCodeReview(c checker.Checker) checker.CheckResult {
 		IsPrReviewRequired,
 		GithubCodeReview,
 		ProwCodeReview,
-		GerritCodeReview,
+		CommitMessageHints,
 	)(c)
 }
 
@@ -147,7 +147,7 @@ func ProwCodeReview(c checker.Checker) checker.CheckResult {
 	return checker.ProportionalResult(totalReviewed, totalMerged, .75)
 }
 
-func GerritCodeReview(c checker.Checker) checker.CheckResult {
+func CommitMessageHints(c checker.Checker) checker.CheckResult {
 	commits, _, err := c.Client.Repositories.ListCommits(c.Ctx, c.Owner, c.Repo, &github.CommitsListOptions{})
 	if err != nil {
 		return checker.RetryResult(err)
@@ -183,6 +183,6 @@ func GerritCodeReview(c checker.Checker) checker.CheckResult {
 	if totalReviewed == 0 {
 		return checker.InconclusiveResult
 	}
-	c.Logf("gerrit code reviews found")
+	c.Logf("code reviews found")
 	return checker.ProportionalResult(totalReviewed, total, .75)
 }

checks/packaging.go

@@ -39,6 +39,10 @@ func Packaging(c checker.Checker) checker.CheckResult {
 		if err != nil {
 			return checker.RetryResult(err)
 		}
+		if fo == nil {
+			// path is a directory, not a file. skip.
+			continue
+		}
 		fc, err := fo.GetContent()
 		if err != nil {
 			return checker.RetryResult(err)