You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Thanks @echedey-ls. Looks like codecov very recently made it easier for us to solve this problem: PRs from forks no longer require a codecov token to upload coverage data. It seems to be working, at least in #2022.
However, coverage also gets uploaded when new commits land on main. From the codecov docs, I think these uploads will not be able to use the "tokenless" route available for PRs. For that, I have added a github secret with a codecov upload token.
With those two changes, I think this problem will be solved. I will proceed with merging #2022 and see if things work as expected on main.
PR uploads may still randomly fail; I don't think there's much we can do about that, at least for now. Re-triggering failed jobs continues to be the workaround.
Describe the bug
Originally from #2006 (comment), but also seen at #2007.
The reported error is related to upload.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
To be uploaded correctly.
Screenshots
![image](https://private-user-images.githubusercontent.com/80125792/322117063-f293d595-adad-4a0d-817a-08e906cfc90b.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTg2MzI5MjMsIm5iZiI6MTcxODYzMjYyMywicGF0aCI6Ii84MDEyNTc5Mi8zMjIxMTcwNjMtZjI5M2Q1OTUtYWRhZC00YTBkLTgxN2EtMDhlOTA2Y2ZjOTBiLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA2MTclMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNjE3VDEzNTcwM1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTIzNDAxYTMzNTE0OWVhZjgyZDZjNzFiNDlmYzA3NWMyN2RhYmEwOWE0MjNiMjZjMmM1OGNiMTAwY2RmNTQxNjUmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.9YAO_AQ0lPNFvCYBsLCe9AZStygWqfwTZvXtGliYcV8)
Versions:
N/A
Additional context
These links may be useful:
Pinging @pvlib/maintainers, I can't be of help with this issue.
The text was updated successfully, but these errors were encountered: