Skip to content

chore: disable automatic PR review trigger#966

Open
jesseturner21 wants to merge 1 commit intoaws:mainfrom
jesseturner21:chore/disable-auto-pr-review
Open

chore: disable automatic PR review trigger#966
jesseturner21 wants to merge 1 commit intoaws:mainfrom
jesseturner21:chore/disable-auto-pr-review

Conversation

@jesseturner21
Copy link
Copy Markdown
Contributor

Summary

  • Comments out the pull_request trigger on the AI review workflow so it no longer runs automatically on PR open/reopen
  • The workflow can still be triggered manually via workflow_dispatch

Test plan

  • Verify the workflow no longer triggers on PR open/reopen
  • Verify manual workflow_dispatch still works

Comment out the pull_request trigger so the AI review workflow
only runs via manual workflow_dispatch.
@jesseturner21 jesseturner21 requested a review from a team April 24, 2026 21:14
@github-actions github-actions Bot added the size/xs PR size: XS label Apr 24, 2026
@github-actions github-actions Bot added the agentcore-harness-reviewing AgentCore Harness review in progress label Apr 24, 2026
@jesseturner21
Copy link
Copy Markdown
Contributor Author

This PR doesn't actually achieve its stated goal — it's effectively a no-op after merging.

The branch was created from a commit before #958 merged to main. PR #958 changed the workflow trigger from pull_request to pull_request_target (for fork PR support). This PR's diff comments out pull_request:, but on current main that line doesn't exist — it's pull_request_target: instead.

I verified this by merging the branch locally: git diff origin/main against the merge result returns empty, so merging this PR would leave pull_request_target: still active and the workflow would keep auto-triggering on PR open/reopen.

To actually disable the automatic trigger, you'll need to rebase onto current main and comment out (or remove) the pull_request_target: block instead:

on:
  # pull_request_target:
  #   types: [opened, reopened]
  workflow_dispatch:
    ...

@github-actions github-actions Bot removed the agentcore-harness-reviewing AgentCore Harness review in progress label Apr 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/xs PR size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant