CLOUD-727: Bump golang.org/x/mod from 0.18.0 to 0.19.0 #2889
reviewdog.yml
on: pull_request
runner / suggester / golangci-lint
1m 43s
runner / suggester / gofmt
8s
runner / suggester / shfmt
12s
runner / shellcheck
5s
runner / misspell
20s
runner / alex
34s
runner / manifests
52s
Annotations
12 errors and 21 warnings
runner / shellcheck
reviewdog: Too many results (annotations) in diff.
You may miss some annotations due to GitHub limitation for annotation created by logging command.
Please check GitHub Actions log console to see all results.
Limitation:
- 10 warning annotations and 10 error annotations per step
- 50 annotations per job (sum of annotations from all the steps)
- 50 annotations per run (separate from the job annotations, these annotations aren't created by users)
Source: https://github.com/orgs/community/discussions/26680#discussioncomment-3252835
|
runner / shellcheck
Process completed with exit code 1.
|
runner / misspell:
.golangci.yaml#L14
[misspell] reported by reviewdog 🐶
"importas" is a misspelling of "imports"
Raw Output:
./.golangci.yaml:14:6: "importas" is a misspelling of "imports"
|
runner / misspell:
.golangci.yaml#L67
[misspell] reported by reviewdog 🐶
"importas" is a misspelling of "imports"
Raw Output:
./.golangci.yaml:67:2: "importas" is a misspelling of "imports"
|
runner / misspell:
internal/controller/runtime/ticker.go#L50
[misspell] reported by reviewdog 🐶
"cancelled" is a misspelling of "canceled"
Raw Output:
./internal/controller/runtime/ticker.go:50:28: "cancelled" is a misspelling of "canceled"
|
runner / misspell:
internal/controller/runtime/ticker.go#L72
[misspell] reported by reviewdog 🐶
"cancelled" is a misspelling of "canceled"
Raw Output:
./internal/controller/runtime/ticker.go:72:24: "cancelled" is a misspelling of "canceled"
|
runner / misspell:
internal/upgradecheck/http.go#L164
[misspell] reported by reviewdog 🐶
"cancelled" is a misspelling of "canceled"
Raw Output:
./internal/upgradecheck/http.go:164:66: "cancelled" is a misspelling of "canceled"
|
runner / misspell:
internal/bridge/client.go#L241
[misspell] reported by reviewdog 🐶
"cancelled" is a misspelling of "canceled"
Raw Output:
./internal/bridge/client.go:241:14: "cancelled" is a misspelling of "canceled"
|
runner / misspell:
internal/bridge/client.go#L296
[misspell] reported by reviewdog 🐶
"cancelled" is a misspelling of "canceled"
Raw Output:
./internal/bridge/client.go:296:30: "cancelled" is a misspelling of "canceled"
|
runner / misspell:
internal/bridge/client.go#L311
[misspell] reported by reviewdog 🐶
"cancelled" is a misspelling of "canceled"
Raw Output:
./internal/bridge/client.go:311:14: "cancelled" is a misspelling of "canceled"
|
runner / misspell:
deploy/cr.yaml#L451
[misspell] reported by reviewdog 🐶
"calulated" is a misspelling of "calculated"
Raw Output:
./deploy/cr.yaml:451:81: "calulated" is a misspelling of "calculated"
|
runner / alex
reviewdog: Too many results (annotations) in diff.
You may miss some annotations due to GitHub limitation for annotation created by logging command.
Please check GitHub Actions log console to see all results.
Limitation:
- 10 warning annotations and 10 error annotations per step
- 50 annotations per job (sum of annotations from all the steps)
- 50 annotations per run (separate from the job annotations, these annotations aren't created by users)
Source: https://github.com/orgs/community/discussions/26680#discussioncomment-3252835
|
runner / shellcheck:
hack/update-pgmonitor-installer.sh#L33
[shellcheck] reported by reviewdog 🐶
Use 'cd ... || exit' or 'cd ... || return' in case cd fails.
Raw Output:
./hack/update-pgmonitor-installer.sh:33:1: warning: Use 'cd ... || exit' or 'cd ... || return' in case cd fails. (ShellCheck.SC2164)
|
runner / shellcheck:
hack/update-pgmonitor-installer.sh#L39
[shellcheck] reported by reviewdog 🐶
Expressions don't expand in single quotes, use double quotes for that.
Raw Output:
./hack/update-pgmonitor-installer.sh:39:12: info: Expressions don't expand in single quotes, use double quotes for that. (ShellCheck.SC2016)
|
runner / shellcheck:
hack/update-pgmonitor-installer.sh#L52
[shellcheck] reported by reviewdog 🐶
Expressions don't expand in single quotes, use double quotes for that.
Raw Output:
./hack/update-pgmonitor-installer.sh:52:8: info: Expressions don't expand in single quotes, use double quotes for that. (ShellCheck.SC2016)
|
runner / shellcheck:
hack/create-kubeconfig.sh#L62
[shellcheck] reported by reviewdog 🐶
$/${} is unnecessary on arithmetic variables.
Raw Output:
./hack/create-kubeconfig.sh:62:12: info: $/${} is unnecessary on arithmetic variables. (ShellCheck.SC2004)
|
runner / shellcheck:
hack/create-todo-patch.sh#L39
[shellcheck] reported by reviewdog 🐶
See if you can use ${variable//search/replace} instead.
Raw Output:
./hack/create-todo-patch.sh:39:26: info: See if you can use ${variable//search/replace} instead. (ShellCheck.SC2001)
|
runner / shellcheck:
hack/create-todo-patch.sh#L42
[shellcheck] reported by reviewdog 🐶
Expressions don't expand in single quotes, use double quotes for that.
Raw Output:
./hack/create-todo-patch.sh:42:91: info: Expressions don't expand in single quotes, use double quotes for that. (ShellCheck.SC2016)
|
runner / shellcheck:
hack/create-todo-patch.sh#L49
[shellcheck] reported by reviewdog 🐶
Expressions don't expand in single quotes, use double quotes for that.
Raw Output:
./hack/create-todo-patch.sh:49:91: info: Expressions don't expand in single quotes, use double quotes for that. (ShellCheck.SC2016)
|
runner / shellcheck:
installers/olm/install.sh#L37
[shellcheck] reported by reviewdog 🐶
Note that A && B || C is not if-then-else. C may run when A is true.
Raw Output:
./installers/olm/install.sh:37:108: info: Note that A && B || C is not if-then-else. C may run when A is true. (ShellCheck.SC2015)
|
runner / shellcheck:
installers/olm/install.sh#L108
[shellcheck] reported by reviewdog 🐶
Note that A && B || C is not if-then-else. C may run when A is true.
Raw Output:
./installers/olm/install.sh:108:70: info: Note that A && B || C is not if-then-else. C may run when A is true. (ShellCheck.SC2015)
|
runner / shellcheck:
installers/olm/install.sh#L122
[shellcheck] reported by reviewdog 🐶
Note that A && B || C is not if-then-else. C may run when A is true.
Raw Output:
./installers/olm/install.sh:122:98: info: Note that A && B || C is not if-then-else. C may run when A is true. (ShellCheck.SC2015)
|
runner / alex:
CONTRIBUTING.md#L21
[alex] reported by reviewdog 🐶
Be careful with `bigger`, it’s profane in some cases bigger retext-profanities
Raw Output:
21:94-21:100 warning Be careful with `bigger`, it’s profane in some cases bigger retext-profanities
|
runner / alex:
LICENSE.md#L136
[alex] reported by reviewdog 🐶
Be careful with `executed`, it’s profane in some cases executed retext-profanities
Raw Output:
136:34-136:42 warning Be careful with `executed`, it’s profane in some cases executed retext-profanities
|
runner / alex:
LICENSE.md#L161
[alex] reported by reviewdog 🐶
Be careful with `failure`, it’s profane in some cases failure retext-profanities
Raw Output:
161:54-161:61 warning Be careful with `failure`, it’s profane in some cases failure retext-profanities
|
runner / alex:
README.md#L16
[alex] reported by reviewdog 🐶
`simple` may be insensitive, try not to use it simple retext-equality
Raw Output:
16:27-16:33 warning `simple` may be insensitive, try not to use it simple retext-equality
|
runner / alex:
docs/content/references/crd.md#L2362
[alex] reported by reviewdog 🐶
Be careful with `execution`, it’s profane in some cases execution retext-profanities
Raw Output:
2362:26-2362:35 warning Be careful with `execution`, it’s profane in some cases execution retext-profanities
|
runner / alex:
docs/content/references/crd.md#L3715
[alex] reported by reviewdog 🐶
`host` may be insensitive, use `presenter`, `entertainer`, `emcee` instead host-hostess retext-equality
Raw Output:
3715:61-3715:65 warning `host` may be insensitive, use `presenter`, `entertainer`, `emcee` instead host-hostess retext-equality
|
runner / alex:
docs/content/references/crd.md#L3717
[alex] reported by reviewdog 🐶
`host` may be insensitive, use `presenter`, `entertainer`, `emcee` instead host-hostess retext-equality
Raw Output:
3717:52-3717:56 warning `host` may be insensitive, use `presenter`, `entertainer`, `emcee` instead host-hostess retext-equality
|
runner / alex:
docs/content/references/crd.md#L3783
[alex] reported by reviewdog 🐶
`host` may be insensitive, use `presenter`, `entertainer`, `emcee` instead host-hostess retext-equality
Raw Output:
3783:46-3783:50 warning `host` may be insensitive, use `presenter`, `entertainer`, `emcee` instead host-hostess retext-equality
|
runner / alex:
docs/content/references/crd.md#L3784
[alex] reported by reviewdog 🐶
`host` may be insensitive, use `presenter`, `entertainer`, `emcee` instead host-hostess retext-equality
Raw Output:
3784:33-3784:37 warning `host` may be insensitive, use `presenter`, `entertainer`, `emcee` instead host-hostess retext-equality
|
runner / alex:
docs/content/references/crd.md#L4020
[alex] reported by reviewdog 🐶
Be careful with `execution`, it’s profane in some cases execution retext-profanities
Raw Output:
4020:26-4020:35 warning Be careful with `execution`, it’s profane in some cases execution retext-profanities
|
runner / suggester / golangci-lint
Restore cache failed: Dependencies file is not found in /home/runner/work/percona-postgresql-operator/percona-postgresql-operator. Supported file pattern: go.sum
|