Lint and Test

Update linting and simplify code #11

Sign in to view logs

Summary
Jobs
- Lint
- Test
Run details
- Usage
- Workflow file

Triggered via push April 18, 2024 18:08

main

Status Failure

Total duration 2m 42s

Artifacts –

lint_test.yaml

on: push

Annotations

1 error and 5 warnings

issues found

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: arduino/setup-task@v1. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: arduino/setup-task@v1. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

Lint: goodreads/client.go#L99

var-naming: method GetBooksByIds should be GetBooksByIDs (revive)

Lint: goodreads/client.go#L156

var-naming: struct field BookIds should be BookIDs (revive)

Lint: server/router.go#L53

unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)