fix: replace manual loop with slices.Contains in isValidWorkflowRunConclusion

[Copilot]

golangci-lint v2.8.0 introduced the modernize/slicescontains rule, which flagged a manual for-loop in isValidWorkflowRunConclusion that should use slices.Contains.

Changes

• pkg/workflow/frontmatter_extraction_yaml.go: Replace manual loop with slices.Contains and add "slices" import

// Before
func isValidWorkflowRunConclusion(v string) bool {
    for _, valid := range validWorkflowRunConclusions {
        if v == valid {
            return true
        }
    }
    return false
}

// After
func isValidWorkflowRunConclusion(v string) bool {
    return slices.Contains(validWorkflowRunConclusions, v)
}

[Copilot]

Pull request overview

Updates isValidWorkflowRunConclusion to satisfy golangci-lint’s modernize/slicescontains by replacing a manual loop with slices.Contains.

Changes:

• Add the standard library slices import.
• Replace the manual for/if membership check in isValidWorkflowRunConclusion with slices.Contains.

Show a summary per file

File	Description
pkg/workflow/frontmatter_extraction_yaml.go	Uses slices.Contains for workflow run conclusion validation and adds the needed import.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

• Files reviewed: 1/1 changed files
• Comments generated: 0