ci: improve CI workflows and add golangci-lint config#71
Merged
liangshuo-1 merged 9 commits intomainfrom Mar 30, 2026
Merged
Conversation
- Add path filters to avoid unnecessary CI runs on non-Go changes - Use go-version-file instead of hardcoded Go version - Unify runners to ubuntu-latest - Consolidate staticcheck/vet into golangci-lint with curated linter set - Add go mod tidy check, govulncheck, and dependency license check - Enable race detector in coverage, increase test timeout to 5m - Add build verification step to tests workflow - Add .codecov.yml with patch coverage target (60%) - Add .golangci.yml (v2) with security and correctness linters Change-Id: I409beb21cc1f1568ff47739c0a00f6214c10a0dd
- Remove Codecov action dependency and CODECOV_TOKEN usage - Generate coverage report using go tool cover and display in Job Summary - Rename job from 'codecov' to 'coverage' - Remove .codecov.yml from paths filter Change-Id: Ib65dab6c4d7117c3300a9ea31eb1550537c72f88
Change-Id: Ic1c492dd339f5460d2be2971ac65ea8f99e524eb
…t restriction Change-Id: I87274abf9780eb8b6350e98a27302ec5acc2a2e5
…ia --new-from-rev Change-Id: I3d4a13cfd7b6c02e4098b04b8533a7248185c077
Change-Id: I112279c5ec06dc0aa3aa7e01d564ea27fbd20533
Change-Id: Iec57e8fbe42699f687d931d9dde2f879f2ae5b02
…blocking - Add exptostd, gocheckcompilerdirectives, gochecksumtype, gomoddirectives linters - Move gosec, staticcheck, errname, errorlint, misspell to TODO for later enablement - Remove G104 exclusion (errcheck is disabled) - Make govulncheck continue-on-error until Go version is upgraded Change-Id: I330ece4f202229aee1e2f50790f6b22738704c05
Change-Id: Ifd018ebe79cd18402171417b1b73313af2d23c6d
MaxHuang22
approved these changes
Mar 30, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
pathsfilters to all workflows to skip unnecessary CI runs on non-Go changesgo-version-file: go.modinstead of hardcoded Go version for single source of truthubuntu-lateststaticcheckandvetjobs intogolangci-lintwith a curated linter setgo mod tidycheck,govulncheck(v1.1.4), andgo-licenses(v2.0.1) dependency license check-racein coverage workflow, increase test timeout from 30s to 5mgo buildverification step to tests workflow.codecov.ymlwith project status as informational and 60% patch coverage target.golangci.yml(v2) with security and correctness focused lintersTest plan
.gofile changesgo mod tidycheck passes