enforce: add stringsindexcontains + stringscountcontains to CI linter batch#45409
Conversation
…ntains+stringscountcontains to CI Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Enforces two existing strings.Contains simplification analyzers in CI.
Changes:
- Replaces a flagged
strings.Countcheck withstrings.Contains. - Enables both analyzers for default and WebAssembly lint runs.
Show a summary per file
| File | Description |
|---|---|
pkg/cli/compile_update_check.go |
Simplifies explicit minor-version detection. |
.github/workflows/cgo.yml |
Adds both analyzers to CI lint flags. |
Review details
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 2/2 changed files
- Comments generated: 0
- Review effort level: Medium
|
Run: https://github.com/github/gh-aw/actions/runs/29325358083
|
Ran
Actions taken: No code changes needed — the branch was already in a merge-ready state. Hand-off: CI must be re-triggered by a maintainer (close/reopen PR, |
Two
strings.Contains-simplification linters (stringsindexcontains,stringscountcontains) were fully implemented and test-verified but absent from the CI enforcement gate.Changes
pkg/cli/compile_update_check.go— Fix the single blockingstringscountcontainsviolation:.github/workflows/cgo.yml— Append-stringsindexcontains -stringscountcontainstoLINTER_FLAGSin both the default andGOOS=js GOARCH=wasmcustom-linter invocations (lines 1208 & 1211).