elastic / elastic/fleet-server
Resolve outstanding `mage check:go` linter findings
- Dominant language
- Go
- Stars
- 113
- Forks
- 117
- Avg merge
- 1d 16h
- Merged PRs (30d)
- 112
Description
### Summary
Resolve the outstanding `golangci-lint` findings reported by `mage check:go` on `main`.
Current findings:
```shell
407 issues:
* errcheck: 18
* gocheckcompilerdirectives: 1
* goconst: 351
* gosec: 12
* misspell: 1
* noctx: 5
* nolintlint: 4
* prealloc: 5
* staticcheck: 10
```
### Criteria for completion
- [ ] All current reported findings are addressed.
- [ ] `mage check:go` exits 0 on `main` (no findings, no panics).
- [ ] No new `//nolint:` annotations introduced unless accompanied by an explanatory comment per the `nolintlint` rule.
Contributor guide
Research direction
Start by running `mage check:go` on `main` and use its output to locate the 407 reported findings across the Go codebase. Address each listed linter category, then rerun the command until it exits 0 without panics or findings. Any new `//nolint:` annotation must include an explanatory comment.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go
- Domain
- tooling
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100