ASC-352 Add proxy tests for components #2907
jkopriva
This run and associated checks have been archived and are scheduled for deletion.
Learn more about checks retention
ci-build.yml
on: pull_request
GolangCI Lint
1m 42s
Unit Tests
1m 55s
Annotations
3 errors and 4 warnings
|
GolangCI Lint:
test/e2e/proxy_test.go#L1136
Error return value of `json.Unmarshal` is not checked (errcheck)
|
|
GolangCI Lint:
testsupport/appstudio/api/v1alpha1/component_types.go#L91
`repostiory` is a misspelling of `repository` (misspell)
|
|
GolangCI Lint
issues found
|
|
GolangCI Lint
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
|
GolangCI Lint:
test/e2e/proxy_test.go#L1037
var-naming: don't use underscores in Go names; method parameter resource_optional should be resourceOptional (revive)
|
|
GolangCI Lint:
test/e2e/proxy_test.go#L1220
unused-parameter: parameter 'skipInitialChecks' seems to be unused, consider removing or renaming it as _ (revive)
|
|
Unit Tests
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|