Fix CI breaking in Forks

[quackzar]

Fixes CI breaking for forks due to insufficient permissions.

[codecov]

Codecov Report

Merging #401 (80c3c9b) into master (5944c37) will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff             @@
##              master      #401   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
  Complexity      3619      3619           
===========================================
  Files            408       408           
  Lines          10210     10210           
  Branches         777       777           
===========================================
  Hits           10210     10210           

Flag	Coverage Δ
unittests	100.00% <ø> (ø)

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

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5944c37...80c3c9b. Read the comment docs.

[jot2re]

Are you sure such extensive permissions are needed?
I don't really know about GitHub actions, but I know that on another project where we are doing the same (code coverage report), it is not needed to set write permissions.
Maybe those yml files can help? https://github.com/TokenScript/attestation/tree/main/.github/workflows

[quackzar]

Are you sure such extensive permissions are needed?
I don't really know about GitHub actions, but I know that on another project where we are doing the same (code coverage report), it is not needed to set write permissions.
Maybe those yml files can help? https://github.com/TokenScript/attestation/tree/main/.github/workflows

I'm not sure, however it seems to be on by default (see here) except in the cases of forks , which is why it fails in the other PR.
The solution I applied here is suggested from the issue #test-reporter/issues/168, however I am not sure if the suggested fix works in our case.

The issue might be solved upstream instead, so we could also disable the test-report either partially for forks or all together.