Handle the case of missing instances when checking for shutdown insta… #22
jofAnnotations
10 errors
|
lint:
internal/instances/instances.go#L112
Error return value is not checked (errcheck)
|
|
lint:
internal/instances/instances.go#L116
Error return value is not checked (errcheck)
|
|
lint:
internal/instances/instances.go#L113
File is not `gofumpt`-ed (gofumpt)
|
|
lint:
internal/instances/instances_test.go#L119
the line is 122 characters long, which exceeds the maximum of 120 characters. (lll)
|
|
lint:
internal/instances/instances.go#L103
`if err != nil` has complex nested blocks (complexity: 5) (nestif)
|
|
lint:
internal/instances/instances.go#L104
comparing with == will fail on wrapped errors. Use errors.Is to check for a specific error (errorlint)
|
|
lint:
internal/instances/instances.go#L121
return with no blank line before (nlreturn)
|
|
lint:
internal/instances/instances.go#L110
error returned from interface method should be wrapped: sig: func (github.com/crusoecloud/crusoe-cloud-controller-manager/internal/client.APIClient).GetInstanceByID(ctx context.Context, instanceID string) (*github.com/crusoecloud/client-go/swagger/v1alpha5.InstanceV1Alpha5, *net/http.Response, error) (wrapcheck)
|
|
lint:
internal/instances/instances.go#L119
error returned from interface method should be wrapped: sig: func (github.com/crusoecloud/crusoe-cloud-controller-manager/internal/client.APIClient).GetInstanceByID(ctx context.Context, instanceID string) (*github.com/crusoecloud/client-go/swagger/v1alpha5.InstanceV1Alpha5, *net/http.Response, error) (wrapcheck)
|
|
lint
issues found
|