Feature/747 fix codecov

[pnoltes]

Hotfix pull request that fixes the codecov upload issue, by using a GitHub codecov token secret.
This is a hotfix and as such will follow the merge then review procedure.

Closes #747

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.20%. Comparing base (cb38185) to head (8b30826).
Report is 32 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #748      +/-   ##
==========================================
+ Coverage   90.17%   90.20%   +0.03%     
==========================================
  Files         223      225       +2     
  Lines       26545    26393     -152     
==========================================
- Hits        23936    23807     -129     
+ Misses       2609     2586      -23     

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

[rlenferink]

per advise of the Apache Infra team, please use the hash/checksum here instead of a tag.

also see 2ca790d were we did this for the previous version

[pnoltes]

Note the coverage also dropped to about 60%. I think is because the test sources are also includes and make coverage excludes the test files for coverage. I will look into this before merging this PR.

[pnoltes]

I am still testing some small changes to see if I can get a cleaner log. After that, I will create a new pull request with a single commit to fix codecov. In my opinion, the small trial and error commits in this PR will unnecessarily clutter the commit log.