Fix the CodeCov Pipeline

[reglim]

The code coverage has had some issues in the past, which are addressed by this change.

Fixes: #1164

Requirements

All new code should be covered with tests, documentation should be updated. CI should pass.

Description of the Change

I changed the go test command to output to the GOCOVERDIR and upgraded the ci to go version 1.20.
I also had to fix some issues that the linter detected, mainly replacing unused parameters with _.

Why this change is important?

The coverage should be accurate and it's not bad to test on the latest go version either.

Checklist

• unit-test added (if change is algorithm)
• functional test added/updated (if change is functional)
• man page updated (if applicable)
• bash completion updated (if applicable)
• documentation updated
• author name in AUTHORS

[randombenj]

LGTM

[codecov]

Codecov Report

Merging #1165 (c49b588) into master (f155ed3) will not change coverage.
The diff coverage is n/a.

❗ Current head c49b588 differs from pull request most recent head 5a2046d. Consider uploading reports for the commit 5a2046d to get more accurate results

@@           Coverage Diff           @@
##           master    #1165   +/-   ##
=======================================
  Coverage   65.96%   65.96%           
=======================================
  Files         143      143           
  Lines       16188    16188           
=======================================
  Hits        10679    10679           
  Misses       4756     4756           
  Partials      753      753           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more