lint: fix nilaway warnings (part 11)

[mmetc]

No description provided.

[github-actions]

@mmetc: There are no area labels on this PR. You can add as many areas as you see fit.

• /area agent
• /area local-api
• /area cscli
• /area appsec
• /area security
• /area configuration

Details

I am a bot created to help the crowdsecurity developers manage community feedback and contributions. You can check out my manifest file to understand my behavior and what I can do. If you want to use this for your project, you can check out the BirthdayResearch/oss-governance-bot repository.

[codecov]

Codecov Report

❌ Patch coverage is 0% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 61.58%. Comparing base (60254d8) to head (07dd4bc).
⚠️ Report is 3 commits behind head on master.

Files with missing lines	Patch %	Lines
cmd/crowdsec-cli/cliitem/cmdinspect.go	0.00%	3 Missing ⚠️
cmd/crowdsec-cli/cliitem/cmdinstall.go	0.00%	1 Missing ⚠️
cmd/crowdsec-cli/cliitem/cmdremove.go	0.00%	1 Missing ⚠️
cmd/crowdsec-cli/cliitem/cmdupgrade.go	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #3910      +/-   ##
==========================================
- Coverage   61.61%   61.58%   -0.04%     
==========================================
  Files         405      405              
  Lines       41782    41789       +7     
==========================================
- Hits        25744    25735       -9     
- Misses      13916    13927      +11     
- Partials     2122     2127       +5     

Flag	Coverage Δ
bats	45.84% <0.00%> (+<0.01%)	⬆️
unit-linux	34.43% <0.00%> (-0.02%)	⬇️
unit-windows	24.33% <0.00%> (-0.02%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[LaurenceJJones]

I dont understand this case, if errors.Is is true that means the err is not nil or is there a case when it can? 🤷🏻

[mmetc]

yes but nilaway doesn't know/handle this automatically so we must be more explicit. it's one of the false positives that is worth to fix to finally enable the linter.