Support exec.shell: for specifying the shell to use
#1730
k1LoWci.yml
on: pull_request
Test
7m 43s
Benchmark
2m 55s
Matrix: Run on each OS
Annotations
11 errors
|
Test:
builtin/diff.go#L13
[gostyle.dontpanic] Don't use panic for normal error handling. Use error and multiple return values. (ref: https://github.com/golang/go/wiki/CodeReviewComments#dont-panic )
|
|
Test:
builtin/json.go#L10
[gostyle.dontpanic] Don't use panic for normal error handling. Use error and multiple return values. (ref: https://github.com/golang/go/wiki/CodeReviewComments#dont-panic )
|
|
Test:
builtin/json.go#L14
[gostyle.dontpanic] Don't use panic for normal error handling. Use error and multiple return values. (ref: https://github.com/golang/go/wiki/CodeReviewComments#dont-panic )
|
|
Test:
builtin/url.go#L10
[gostyle.dontpanic] Don't use panic for normal error handling. Use error and multiple return values. (ref: https://github.com/golang/go/wiki/CodeReviewComments#dont-panic )
|
|
Test:
cdp.go#L26
[gostyle.contexts] Don't add a Context member to a struct type; instead add a ctx parameter to each method on that type that needs to pass it along. The one exception is for methods whose signature must match an interface in the standard library or in a third party library. (ref: https://github.com/golang/go/wiki/CodeReviewComments#contexts ): ctx
|
|
Test:
operator.go#L862
[gostyle.dontpanic] Don't use panic for normal error handling. Use error and multiple return values. (ref: https://github.com/golang/go/wiki/CodeReviewComments#dont-panic )
|
|
Test:
option.go#L819
[gostyle.dontpanic] Don't use panic for normal error handling. Use error and multiple return values. (ref: https://github.com/golang/go/wiki/CodeReviewComments#dont-panic )
|
|
Test:
option.go#L820
[gostyle.dontpanic] Don't use panic for normal error handling. Use error and multiple return values. (ref: https://github.com/golang/go/wiki/CodeReviewComments#dont-panic )
|
|
Test:
ssh.go#L127
[gostyle.dontpanic] Don't use panic for normal error handling. Use error and multiple return values. (ref: https://github.com/golang/go/wiki/CodeReviewComments#dont-panic )
|
|
Test:
ssh.go#L139
[gostyle.dontpanic] Don't use panic for normal error handling. Use error and multiple return values. (ref: https://github.com/golang/go/wiki/CodeReviewComments#dont-panic )
|
|
Test
Process completed with exit code 2.
|