on main help sign in
chore: fix lint on interp.ExecHandler (#3200)
commit6b68930b2d6a9d99b7bd7fe1a84ddf1e8498f475 authorKieran Klukas <me@dunkirk.sh> dateWed 24 Jun 2026, 15:30 (4 months ago) parent7f7eb60
2 files changed+7−1
.golangci.yml+4 internal/shell/run.go+3−1
.golangci.yml+4
@@ -37,6 +37,10 @@ linters:
3737 - text: '(slog|log)\.\w+'
3838 linters:
3939 - noctx
40+ - path: internal/shell/run.go
41+ text: "SA1019.*interp.ExecHandler"
42+ linters:
43+ - staticcheck
4044issues:
4145 max-issues-per-linter: 0
4246 max-same-issues: 0
internal/shell/run.go+3−1
@@ -210,7 +210,9 @@ func execHandlerOption(blockFuncs []BlockFunc) interp.RunnerOption {
210210 for _, mw := range slices.Backward(standardHandlers(blockFuncs)) {
211211 handler = mw(handler)
212212 }
213− return interp.ExecHandler(handler) //nolint:staticcheck // ExecHandlers always appends DefaultExecHandler which lacks process isolation.
213+ // ExecHandlers always appends DefaultExecHandler which lacks process
214+ // group isolation, so we use the deprecated ExecHandler instead.
215+ return interp.ExecHandler(handler)
214216}
215217
216218// nonInteractiveEnvVars are forced on every shell execution to prevent