---

name: go-vet description: Fix go vet warnings

go vet

Built-in static analyzer that catches common mistakes.

Usage

Copygo vet ./...
go vet ./pkg/...

Common Warnings

Printf Format Mismatch

Copy// Bad
fmt.Printf("%d", "string")

// Good
fmt.Printf("%s", "string")

Unreachable Code

Copy// Bad
return value
fmt.Println("never runs")

// Good
fmt.Println("runs")
return value

Composite Literal Uses Unkeyed Fields

Copy// Bad
Person{"Alice", 30}

// Good
Person{Name: "Alice", Age: 30}

Nil Dereference

Copy// Bad
var p *int
fmt.Println(*p)

// Good
if p != nil {
    fmt.Println(*p)
}

Suspicious Mutex Usage

Copy// Bad - mutex copied
func process(mu sync.Mutex) {
    mu.Lock()
}

// Good - pass pointer
func process(mu *sync.Mutex) {
    mu.Lock()
}

Lost Context

Copy// Bad
ctx := context.TODO()

// Good
ctx := context.Background()
// or accept ctx as parameter

Fix All Issues

Copygo vet ./... 2>&1 | tee vet.log