Linters: enable gocyclo, errcheck and gosec
Metadata
Current evaluation
Enable gocyclo, errcheck, and gosec linters in golangci-lint config and fix resulting errors. Triaged but inactive for ~9 months.
Suggested action: keep open
Reason: The issue is labeled 'Triaged' and authored by a maintainer (upils), indicating it was acknowledged as valid. However, it has been inactive for 277 days with only an auto-generated Jira ticket comment. It should remain open as a legitimate enhancement but may need re-prioritization or a maintainer to pick it up.
Impact:
35
Quick Win:
15.75
Staleness:
75
Complexity:
55
Confidence:
70
Support Request:
10
Issue body
### What needs to get done
Enable gocyclo, errcheck and gosec in the the golangci-lint configuration.
Tune the configuration to be consistent with other projects.
Solve returnes errors.
### Why it needs to get done
These linters currently return many errors but are part of the "reasonable" set of linters we want to enable.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b |
Impact:
35
Quick Win:
15.75
Staleness:
75
Complexity:
55
Confidence:
70
Support Request:
10
|
keep open | Enable gocyclo, errcheck, and gosec linters in golangci-lint config and fix resulting errors. Triaged but inactive for ~9 months. |
Update history
| Date | Change |
|---|---|
| created |
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #545 ci: enable gocyclo, errcheck and gosec linters | fetch-service | closed | Enabling gocyclo, errcheck, and gosec linters closed without merging. CI passed, but a reviewer requested reverting specific defer patterns. Misunderstood feedback left changes unresolved, resulting in closure. | |
| #494 lint: Set a linter conf and fix errors | fetch-service | merged | Merged following reviewer approval and successful CI checks. Integrated golangci-lint configuration and resolved linting errors across 117 files, establishing automated static analysis for the codebase. |