Skip to content
This repository was archived by the owner on Jun 2, 2023. It is now read-only.

Commit 93bb034

Browse files
committed
fix builds
1 parent 82a2bfd commit 93bb034

File tree

3 files changed

+6
-5
lines changed

3 files changed

+6
-5
lines changed

.circleci/config.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ jobs:
66
build:
77
docker:
88
# specify the version
9-
- image: circleci/golang:1.9
9+
- image: circleci/golang:1.11
1010
- image: redis
1111

1212
# Specify service dependencies here if necessary

.golangci.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,3 +21,6 @@ linters:
2121
- prealloc
2222
- dupl # temporary
2323
- gosec
24+
- scopelint
25+
- gochecknoglobals
26+
- gochecknoinits

app/lib/executors/shell.go

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -138,8 +138,6 @@ func (s shell) runAsync(ctx context.Context, name string, args ...string) (int,
138138
return 0, nil, nil, err
139139
}
140140

141-
return cmd.Process.Pid, outReader, func() error {
142-
// XXX: it's important to not change error here, because it holds exit code
143-
return cmd.Wait()
144-
}, nil
141+
// XXX: it's important to not change error here, because it holds exit code
142+
return cmd.Process.Pid, outReader, cmd.Wait, nil
145143
}

0 commit comments

Comments
 (0)