Skip to content

docs(contributing): note fork PRs skip Agentic PR review check#120

Merged
leo-aa88 merged 1 commit into
mainfrom
docs/issue-95-contributing-fork-pr-review
May 29, 2026
Merged

docs(contributing): note fork PRs skip Agentic PR review check#120
leo-aa88 merged 1 commit into
mainfrom
docs/issue-95-contributing-fork-pr-review

Conversation

@leo-aa88
Copy link
Copy Markdown
Member

Summary

  • Adds an Agentic PR review (fork and first-time contributors) subsection to CONTRIBUTING.md.
  • Explains why fork-originated PRs do not show the automated review check (same-repo head branch + no fork secrets).
  • Notes the first-time workflow introduction case and that maintainers can validate from an in-repo branch.
  • Links to docs/GITHUB_ACTIONS.md for permissions, secrets, and full fork PR details.

Closes #95

Test plan

  • Doc-only change; no Go code modified.
  • Verified links resolve to .github/workflows/agentctl-pr-review.yml and docs/GITHUB_ACTIONS.md.
  • Content aligns with workflow if: guard and existing GITHUB_ACTIONS.md fork PR sections.

Made with Cursor

External contributors often miss the automated review workflow on fork
PRs because the job requires same-repo head and repository secrets.
Document the behavior and point to GITHUB_ACTIONS.md for full details.

Closes #95

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions
Copy link
Copy Markdown

ReviewGate [PASS] PASS

Stats

  • Files changed: 1
  • Raw LOC changed: 22
  • LOC after §10.4 exclusions (human_loc_changed): 22
  • PR author class: human (human collaborator account) — login leo-aa88 (§10.4.2).

No deterministic warnings fired.

Suggested labels: reviewability-pass

File categories: 1 files (0 risky)

@leo-aa88 leo-aa88 merged commit ec5a0fe into main May 29, 2026
1 check passed
@leo-aa88 leo-aa88 deleted the docs/issue-95-contributing-fork-pr-review branch May 29, 2026 03:44
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.

Docs: CONTRIBUTING note — fork PRs skip Agentic PR review check

1 participant