jonathanmetzman
jonathanmetzman
Thanks. I'm not an expert but it sounds to me like there are good reasons for keeping this check and for turning it off by default. Given this, I think...
Haha, I think the case for disabling it is stronger than the case for enabling it, but I'm probably not going to change it because of inertia. I don't think...
We've agreed internally we will merge your PR David. Thank you everyone for correcting OSS-Fuzz on this. I'll try to merge it this week.
Thank you, I'm going to take a look at this tomrrow.
Sorry, i've been busy lately. I'll actually review this tomorrow.
This is a tricky question, because as @evverx points out, behaving correctly (failing when we can't check out the build) here will actually create more annoyance for our users. I'm...
I think I'm going to if protect the entire action so that it doesn't run at all on forks.
> I think I'm going to if protect the entire action so that it doesn't run at all on forks. Let me think about this again though, as I've mostly...
> > I think I'm going to if protect the entire action so that it doesn't run at all on forks. > > Let me think about this again though,...
Sorry for the delay in fixing this. Could you test if https://github.com/google/oss-fuzz/pull/10967 works @hardik05