search

coderabbitai / ai-pr-reviewer

AI-based Pull Request Summarizer and Reviewer with Chat Capabilities.
https://coderabbit.ai
MIT License
1.48k stars 283 forks source link

Failed runs on deleted branches #224

Closed ckcr4lyf closed 1 year ago

ckcr4lyf commented 1 year ago

Full disclosure: I was explicitly trying to trigger edge cases

I commented on a diff in an MR that was already merged (and hence original branch deleted): https://github.com/fluxninja/openai-pr-reviewer/pull/218/files/362ff2b8e3614a38c5705426672bf9130bdd4b11#diff-6237e411e4b5750f5e3713c4739039620a279cdf9c8c596d573a97e87f1c8e0f

Seems the action tried to initiate a run to review the code based on my comment, but there is no branch, so it failed: https://github.com/fluxninja/openai-pr-reviewer/actions/runs/4762183914/jobs/8464139310#step:2:57

Just reporting a bug!

harjotgill commented 1 year ago

The actions/checkout@v3 step fails, and there isn't much we can do about it. This action isn't even getting invoked. The GitHub event trigger doesn't contain sufficiently filter out this scenario. See PR review comment trigger docs.