ci: explicitly specify token for codecov

SciML / StructuralIdentifiability.jl

Fast and automatic structural identifiability software for ODE systems

https://docs.sciml.ai/StructuralIdentifiability/stable/

MIT License

112 stars 17 forks source link

ci: explicitly specify token for codecov #285

Closed thazhemadam closed 8 months ago

thazhemadam commented 8 months ago

Explicitly specify the codecov token to be used (i.e., CODECOV_TOKEN), given that the latest v4 release of the codecov action requires it to be able to generate coverage reports. Additionally, fail CI if coverage reporting fails, since coverage is an important enough metric for us to ensure that it's tracked consistently.

pogudingleb commented 8 months ago

I have to say that I am not entirely happy about merging the PR to master without review/approval.

ChrisRackauckas commented 8 months ago

This was just a CI update done to all repos because the v4 now requires tokens, so there's a org-wide SSH token that was added to fix codecov usage. I'm not entirely sure how this would be done non-centrally since it would require setting the secrets.