Don't try to upload coverage if CI is run via workflow dispatch on a fork

python-attrs / cattrs

Composable custom class converters for attrs, dataclasses and friends.

https://catt.rs

MIT License

799 stars 111 forks source link

Don't try to upload coverage if CI is run via workflow dispatch on a fork #385

Closed AlexWaygood closed 1 year ago

AlexWaygood commented 1 year ago

While investigating https://github.com/python/typing_extensions/issues/230 and preparing #384, I tried running cattrs's CI on my fork via workflow-dispatch. This was made a little more complicated by the fact that the "upload coverage to codecov" step always seems to fail when the workflow is run on a fork

Tinche commented 1 year ago

Thanks! I want to get rid of codecov long-term but still looking for a good alternative.