Continuous Integration

Merge pull request #89 from nginxinc/NLB-4932-header #220

Sign in to view logs

Triggered via push May 30, 2024 20:25

xynicole

main

Status Failure

Total duration 38s

Artifacts –

cicd.yml

on: push

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)

issues found

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-go@v4, golangci/golangci-lint-action@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

Linting: util_test.go#L14

dot-imports: should not use dot imports (revive)