Skip to content

Request Copilot and ReviewHog reviews on new posthog/posthog PRs - #213

Merged
haacked merged 1 commit into
mainfrom
haacked/hog-ql-and-copilot
Sep 17, 2026
Merged

haacked merged 1 commit into
mainfrom
haacked/hog-ql-and-copilot

Conversation

@haacked

@haacked haacked commented Sep 17, 2026

Copy link
Copy Markdown
Owner
  • create-pr gains a Step 10: after it opens a new PR whose URL is under github.com/PostHog/posthog/, it runs gh pr edit <pr-url> --add-label reviewhog --add-reviewer "@copilot" under the user's gh login. Updates to an existing PR skip it, and so does a PR that Step 9 labeled skip-agent-review.
  • The request is a separate call after creation, so a failed label or reviewer request leaves the PR in place and is reported next to its URL.
  • go folds an outstanding test-gap commit in before Step 6 opens the PR, so the ReviewHog round that create-pr now triggers reviews a head that includes the tests. Step 7's label add on posthog/posthog becomes the documented safe re-add.

Test plan:

https://claude.ai/code/session_01Be5RB9C731bGXr9AEjqfM9

create-pr adds the reviewhog label and requests a Copilot review right after it opens a PR on posthog/posthog, as the user's gh login. The go skill folds an outstanding test-gap commit in before it opens the PR, so the ReviewHog round that create-pr triggers reviews a head that includes the tests.

Claude-Session: https://claude.ai/code/session_01Be5RB9C731bGXr9AEjqfM9
@haacked
haacked marked this pull request as ready for review September 17, 2026 19:37
@haacked
haacked merged commit 0f81e23 into main Sep 17, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant