go-vet

name	go-vet
description	Fix go vet warnings

go vet

Built-in static analyzer that catches common mistakes.

Usage

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

Common Warnings

Printf Format Mismatch

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

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

Unreachable Code

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

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

Composite Literal Uses Unkeyed Fields

// Bad
Person{"Alice", 30}

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

Nil Dereference

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

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

Suspicious Mutex Usage

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

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

Lost Context

// Bad
ctx := context.TODO()

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

Fix All Issues

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