Skip to content

Merge pull request #101 from dengsh12/NLB5024 #310

Merge pull request #101 from dengsh12/NLB5024

Merge pull request #101 from dengsh12/NLB5024 #310

Triggered via push July 10, 2024 19:33
Status Failure
Total duration 2m 34s
Artifacts

cicd.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

4 errors and 2 warnings
Linting: buffer.go#L17
interface method Create must have named param for type string (inamedparam)
Linting: util.go#L67
Magic number: 2, in <operation> detected (mnd)
Linting: types_test.go#L43
expected-actual: need to reverse actual and expected values (testifylint)
Linting
issues found
Linting
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3, actions/setup-go@v4, golangci/golangci-lint-action@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Linting: util_test.go#L14
dot-imports: should not use dot imports (revive)