lint: Drop paralleltest

Enabling t.Parallel for all tests is not a good idea. In the future, limit it to slow tests only.
abhinav · Oct 15, 2023 · 13d7b46 · 13d7b46
1 parent 4999223
commit 13d7b46
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 3 deletions.
diff --git a/.golangci.yml b/.golangci.yml
@@ -15,7 +15,6 @@ linters:
   enable:
     - gofumpt
     - nolintlint
-    - paralleltest
     - revive
 
 linters-settings:

diff --git a/ast_test.go b/ast_test.go
@@ -40,7 +40,6 @@ func TestBlock(t *testing.T) {
 		assert.True(t, b.IsRaw())
 	})
 
-	//nolint:paralleltest // hijacks stdout
 	t.Run("Dump", func(t *testing.T) {
 		stdout, closeStdout := hijackStdout(t)
 
@@ -69,7 +68,6 @@ func TestScript(t *testing.T) {
 		assert.True(t, sb.IsRaw())
 	})
 
-	//nolint:paralleltest // hijacks stdout
 	t.Run("Dump", func(t *testing.T) {
 		stdout, closeStdout := hijackStdout(t)