brainglobe / brainglobe-workflows

Workflows that utilise BrainGlobe tools to perform data analysis and visualisation.
BSD 3-Clause "New" or "Revised" License
9 stars 2 forks source link

Add codecov token #91

Closed K-Meech closed 5 months ago

K-Meech commented 5 months ago

Description

What is this PR

Why is this PR needed? Codecov is currently complaining on all actions runs with: secret-codecov-token is not set. This will be required in the future. See https://docs.codecov.com/docs/quick-start#step-2-get-the-repository-upload-token on how to get a token. and Codecov: Failed to properly create commit: The process 'D:\a\_actions\codecov\codecov-action\v4\dist\codecov.exe' failed with exit code 1

What does this PR do? This PR ensures test_and_deploy is using the codecov token, as recommended in the readme for the neuroinformatics unit test action

References

None

How has this PR been tested?

Hopefully codecov won't error on the github actions runs for this PR

Is this a breaking change?

No

Does this PR require an update to the documentation?

No

Checklist:

codecov[bot] commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 79.43%. Comparing base (cd14205) to head (0b2f39a). Report is 4 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #91 +/- ## ======================================= Coverage 79.43% 79.43% ======================================= Files 18 18 Lines 924 924 ======================================= Hits 734 734 Misses 190 190 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

K-Meech commented 5 months ago

Can confirm - this removed all codecov errors on the last run :tada: