Skip to content

fix: enable E2E report comment for labeled trigger#80

Merged
Kavirubc merged 1 commit intomainfrom
fix/e2e-report-trigger
Feb 18, 2026
Merged

fix: enable E2E report comment for labeled trigger#80
Kavirubc merged 1 commit intomainfrom
fix/e2e-report-trigger

Conversation

@Kavirubc
Copy link
Member

@Kavirubc Kavirubc commented Feb 18, 2026

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 using pull_request_target.

Updated the condition to match pull_request_target so the bot posts the test results.

Summary by CodeRabbit

  • Chores
    • Updated the GitHub Actions workflow trigger for automated PR comments to respond to pull request target events instead of pull request review events.

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>
@coderabbitai
Copy link

coderabbitai bot commented Feb 18, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

A GitHub Actions workflow condition was modified to trigger the PR comment step on pull_request_target instead of pull_request_review, shifting the event that initiates the final comment posting. The workflow logic and outputs remain unchanged.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow
.github/workflows/e2e-test.yml
Modified PR-comment step condition to trigger on pull_request_target event instead of pull_request_review event.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Suggested labels

bug, ci/cd

Suggested reviewers

  • gh-simili-bot

Poem

🐰 A workflow leaps to a different beat,
From review to target, the timing's neat!
The comment still sings with the same cheer,
Just when it posts—ah, a shift appears!
Carrots for CI/CD, hooray! 🥕✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/e2e-report-trigger

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Kavirubc Kavirubc merged commit 7fa7fea into main Feb 18, 2026
5 of 6 checks passed
@Kavirubc Kavirubc deleted the fix/e2e-report-trigger branch February 18, 2026 16:36
@gh-simili-bot
Copy link
Contributor

Simili Triage Report

Note

Quality Score: 8.5/10 (Good)
The issue could be improved. See suggestions below.

Classification

Category Value
Labels
Quality Improvements
  • Missing direct link to a workflow run demonstrating the skipped comment
  • Provide a link to a specific GitHub Actions workflow run where the report comment was observed to be skipped.
Similar Threads
Similarity Thread Status
85% #79 fix: use 'e2e' label trigger for secure secrets... Open
79% #74 fix: prevent bot loop & enable E2E for forks Closed
78% #78 fix: run E2E only when review is approved (secu... Open

Generated by Simili Bot

@gh-simili-bot gh-simili-bot added the bug Something isn't working label Feb 18, 2026
Sachindu-Nethmin pushed a commit to Sachindu-Nethmin/simili-bot that referenced this pull request Feb 19, 2026
…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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments