PR Agent skips review on synchronize after new commit (“Skipping action: synchronize”) #2079
|
Hello, I noticed PR Agent sometimes skips reviewing a PR after I push a new commit and I'm not quite sure why this would be the case or how PR Agent makes this decision. The GitHub Actions log shows: Expected Actual Steps to Reproduce
Workflow Config on:
pull_request:
types: [opened, reopened, ready_for_review, synchronize]
issue_comment:
types: [created]
workflow_dispatch:
jobs:
pr_agent:
if: ${{ github.event.sender && github.event.sender.type != 'Bot' }}
runs-on: ubuntu-latest
steps:
- name: qodo-ai/pr-agent Action
id: pragent
uses: qodo-ai/pr-agent@eb4cdbb115dfb711375a24eccc75e3ef73ec03bc
env:
[redacted environment variables]
github_action_config.auto_describe: true
github_action_config.auto_improve: true
github_action_config.auto_review: trueQuestions
Thanks! I'm starting out with a Discussion to make sure I'm understanding the problem correctly before creating an Issue for it. |
Replies: 3 comments
|
Hi @jeffschaper , thank you so much for adding those questions. |
|
Your reading was right, and the skip is deliberate rather than a fault. The Action's |
Your reading was right, and the skip is deliberate rather than a fault. The Action's
github_action_config.pr_actionslist defaults to opened, reopened, ready_for_review and review_requested, so asynchronizeevent matches nothing and falls through to that log line; listingsynchronizein the workflowtypes:is necessary but not sufficient. There are two supported ways to get a review on each new commit. Either addsynchronizetogithub_action_config.pr_actions, which reruns the same auto tools, or setgithub_action_config.handle_push_trigger: truetogether withpush_commandsto run a lighter set on pushes. With the push trigger, merge commits and bot commits are ignored by default viapus…