golang/go

x/tools/go/analysis/passes/modernize: add testingsetenv analyzer

Open

Aperta il 22 feb 2026

Vedi su GitHub
 (13 commenti) (4 reazioni) (1 assegnatario)Go (133.883 star) (19.008 fork)batch import
FeatureRequestProposalToolshelp wanted

Descrizione

Proposal Details

Add a new analyzer that replaces manual environment variable save/restore with t.Setenv (Go 1.17+).

// Before
defer os.Setenv("GOOS", os.Getenv("GOOS"))
os.Setenv("GOOS", f[0])

// After
t.Setenv("GOOS", f[0])

Conditions

  • Pattern: defer os.Setenv(key, os.Getenv(key)) followed by os.Setenv(key, value)
  • defer must precede os.Setenv (defer args are evaluated immediately)
  • Within test function, before any t.Parallel() call
  • Go 1.17+

Similar to testingcontext analyzer.

Guida contributor

Tech stack
go
Dominio
tooling
Tipo issue
feature
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 days
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
active
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
clear
Prerequisiti
Gostatic analysis
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65
Direzione di ricerca
Studia gli analizzatori modernize esistenti (es. testingcontext) nel repository x/tools. Comprendi il pattern matching per defer os.Setenv seguito da os.Setenv e implementa un nuovo analizzatore usando lo stesso framework di analisi.