Do not upload coverage results to Codecov when run in a fork #496
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When I sync my fork (eg today in jdblischak@98f1234), the test-coverage workflow fails because I don't have
CODECOV_TOKEN
set.We only need coverage uploads to Codecov to run on the main repository, so this PR updates the workflow to skip the upload step when running on a fork. However, when running in a PR submitted from a fork, it still performs token-less uploads to Codecov.
xref: #489, Merck/simtrial#307, keaven/gsDesign#179