Fix Linting Issues in CLI Tests · Entire
can you fix linting
1mo ago·1m·4 calls
Lint passes — 0 issues.
The failure was a gocritic exitAfterDefer warning in cmd/entire/cli/global_test.go: a defer os.RemoveAll(isolationDir) that could never run because TestMain ends with os.Exit(code). Since the same cleanup already runs explicitly on the line right before os.Exit, I removed the dead defer.
can you commit and push
1mo ago·31.5s·2 calls
Committed and pushed to soph/cli-testmain-token-isolation.
[28f560acli: drop dead defer in test TestMain that os.Exit skips
The deferred os.RemoveAll never ran because TestMain ends with os.Exit(code); the same cleanup already runs explicitly just before the exit. Removing it clears the gocritic exitAfterDefer lint failure.
Co-Authored-By: Claude Opus 4.8 (1M context) noreply@anthropic.com Entire-Checkpoint: 438e4b46f73c-3](/content/gh/entireio/cli/commit/28f560a368f71269651197a59dafbfb020d95291/index.html)