fix: Address new linter issues #244
kinbikogo.yml
on: push
Build and cache dependencies
5s
Run tests
28s
Run tests with race checker
20s
linting
22s
Annotations
10 errors and 4 warnings
|
linting:
integration_test.go#L24
unused-parameter: parameter 'w' seems to be unused, consider removing or renaming it as _ (revive)
|
|
linting:
integration_test.go#L33
unused-parameter: parameter 'w' seems to be unused, consider removing or renaming it as _ (revive)
|
|
linting:
builds/http.go#L40
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"publisher created incorrectly; please use NewPublisher or DefaultPublisher to construct your builds.Publisher\")" (err113)
|
|
linting:
builds/http.go#L79
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"error when sending message: %s\", res.Errors[0])" (err113)
|
|
linting:
builds/validation.go#L13
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(`APIKey must be 32 hex characters, but got \"%s\"`, r.APIKey)" (err113)
|
|
linting:
builds/validation.go#L17
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(`AppVersion must be present`)" (err113)
|
|
linting:
builds/validation.go#L25
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(`SourceControl.Repository must be present when SourceControl is set`)" (err113)
|
|
linting:
builds/validation.go#L28
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(`SourceControl.Revision must be present when SourceControl is set`)" (err113)
|
|
linting:
builds/validation.go#L45
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(`SourceControl.Provider must be unset (for automatic inference) or one of %v`, validProviders)" (err113)
|
|
linting:
configuration.go#L93
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(`API key must be 32 hex characters, but got \"%s\"`, cfg.APIKey)" (err113)
|
|
Build and cache dependencies
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-go@v4. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
Run tests with race checker
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-go@v4. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
linting
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-go@v4. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
Run tests
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-go@v4. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|