{CI only} Use pull_request_target instead of pull_request

nats-io / nats.c

A C client for NATS

Apache License 2.0

382 stars 132 forks source link

{CI only} Use pull_request_target instead of pull_request #767

Closed levb closed 2 months ago

levb commented 2 months ago

We are having an issue running coverage upload from forked repo PRs. This should solve it, I think.

Testing for now from my own fork.

levb commented 2 months ago

@kozlovic I am still experimenting with this one

kozlovic commented 2 months ago

@levb Sorry, I thought that I was asked to review, my mistake.

levb commented 2 months ago

I am just going to take codecov out of the forked repo PR flow for now. It appears that the codecov people are working on the issue, and there is a workaround but it's kludgy. Will close this PR and open another one to remove codecov from fork PRs for now