codecov-action gha with token

[bjlittle]

🚀 Pull Request

Description

This pull-request explicitly uses the CODECOV_TOKEN with the codecov-action to upload a test report.

Speculating whether this assists with #5915

---

Consult Iris pull request check list

---

Add any of the below labels to trigger actions on this PR:

• benchmark_this Request that this pull request be benchmarked to check if it introduces performance shifts

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.78%. Comparing base (7c313ff) to head (d16c1f4).
Report is 27 commits behind head on main.

❗ Current head d16c1f4 differs from pull request most recent head 004faac. Consider uploading reports for the commit 004faac to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5941      +/-   ##
==========================================
+ Coverage   89.76%   89.78%   +0.01%     
==========================================
  Files          93       93              
  Lines       22982    23007      +25     
  Branches     5006     5017      +11     
==========================================
+ Hits        20630    20657      +27     
+ Misses       1622     1620       -2     
  Partials      730      730              

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

[bjlittle]

Migrating to explicitly using the token successfully uploads the test report i.e., the secrets.CODECOV_TOKEN is valid 👍

However, I noticed that we appear to be uploading 6 test reports instead of 1 😮

Hmmm I'll confirm whether the action is uploading all of those files or just the single coverage.xml ...

[bjlittle]

Okay, I'm not going to burn any more time on this ...

You can use the file: or files: arguments to the gha, but that doesn't seem to make any difference to the action detecting 6 files.

Using the verbose: flag to the gha doesn't offer up any more information on how many files are actually uploaded to the codecov server, and there's nothing obvious on the codecov server-side to indicate the number of report files uploaded.

The fact that this is "working" seems sufficient ... however, whether this change addresses #5915 is yet to be determined.

[pp-mo]

Despite your reservations, I'm happy that this is at least going in the right direction.
So let's just see what happens.

[bjlittle]

@pp-mo Awesome, thanks 🍻