fix: enable E2E report comment for labeled trigger#80
Conversation
The 'Comment on PR' step was skipped because it checked for `pull_request_review`, but the trigger was changed to `pull_request_target`. Updating condition to ensure reports are posted. Signed-off-by: Kavirubc <hapuarachchikaviru@gmail.com>
|
Caution Review failedThe pull request is closed. 📝 WalkthroughWalkthroughA GitHub Actions workflow condition was modified to trigger the PR comment step on Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Simili Triage ReportNote Quality Score: 8.5/10 (Good) Classification
Quality Improvements
Similar Threads
Generated by Simili Bot |
…iligh#80) The 'Comment on PR' step was skipped because it checked for `pull_request_review`, but the trigger was changed to `pull_request_target`. Updating condition to ensure reports are posted. Signed-off-by: Kavirubc <hapuarachchikaviru@gmail.com> Signed-off-by: Sachindu-Nethmin <sachindunethminweerasinghe@gmail.com>
Fix
The E2E workflow ran successfully on PRs, but the Report Comment was skipped.
Why? The step condition checked for
pull_request_review(old trigger), but we are now usingpull_request_target.Updated the condition to match
pull_request_targetso the bot posts the test results.Summary by CodeRabbit