fix unit test #2294
LordofAvernus
This run and associated checks have been archived and are scheduled for deletion.
Learn more about checks retention
Annotations
10 errors and 4 warnings
|
lint:
sqle/model/instance.go#L327
Consider preallocating `instanceIds` (prealloc)
|
|
lint:
sqle/api/middleware/jwt_test.go#L37
`reponse` is a misspelling of `response` (misspell)
|
|
lint:
sqle/api/middleware/jwt_test.go#L45
`reponse` is a misspelling of `response` (misspell)
|
|
lint:
sqle/dms/instance.go#L129
unnecessary conversion (unconvert)
|
|
lint:
sqle/dms/instance.go#L295
S1019: should use make(map[uint64]string) instead (gosimple)
|
|
lint:
sqle/api/controller/v1/statistic_ce.go#L20
`getLicenseUsageV1` is unused (deadcode)
|
|
lint:
sqle/api/controller/v1/task.go#L725
`getDistinctInstNameList` is unused (deadcode)
|
|
lint:
sqle/api/controller/v1/task.go#L174
unnecessary conversion (unconvert)
|
|
lint:
sqle/api/controller/v1/task.go#L705
unnecessary conversion (unconvert)
|
|
lint
issues found
|
|
lint
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-go@v2, golangci/golangci-lint-action@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
|
lint
The `save-state` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
lint
The `save-state` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
test
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-go@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|