Skip to content

Assign PR author and generalize the devin workflow#7970

Open
andrew-polk wants to merge 1 commit into
masterfrom
devin-pr-automation
Open

Assign PR author and generalize the devin workflow#7970
andrew-polk wants to merge 1 commit into
masterfrom
devin-pr-automation

Conversation

@andrew-polk

@andrew-polk andrew-polk commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Builds on top of #7958.

What

  • Renames run-devin.ymlpr-automation.yml and changes the workflow name from run-devin to PR Automation, since it now does more than just run Devin.
  • Adds a step that assigns each new/updated PR to its author, so the author shows up on the PR Review Tracker board.

Why here (not in the tracker workflow)

The PR Review Tracker workflow only carries PROJECT_TOKEN (org-Projects scope, not repo writes). This workflow already declares pull-requests: write and uses the default github.token, so assignment needs no new secret or permission.

Notes

  • The assign step runs first, so a failure in the later (flaky) Playwright/browser step can't cause the assignment to be skipped.
  • Fork PRs are still skipped (no writable token), same as before.

🤖 Generated with Claude Code

Devin review


This change is Reviewable

Rename run-devin.yml to pr-automation.yml and broaden its scope: in
addition to kicking off a devinreview.com review, it now assigns each new
PR to its author so the author is visible on the PR Review Tracker board.

The assign step runs first and uses the default github.token (the job
already declares pull-requests: write), so it needs no extra secret and
won't be skipped if the later browser-driven devin step fails.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@andrew-polk andrew-polk marked this pull request as draft June 16, 2026 18:41
@greptile-apps

greptile-apps Bot commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR renames run-devin.ymlpr-automation.yml and inserts a new first step that assigns each non-fork PR to its author via gh pr edit --add-assignee, enabling the author to appear on the PR Review Tracker board.

  • The assign step uses environment variables for all GitHub context values (PR number, author login, repo), avoiding shell/YAML injection risks, and relies on the existing pull-requests: write permission with the default github.token — no new secrets needed.
  • Placing the step first ensures that the flaky Playwright/browser step cannot prevent author assignment from running.
  • Re-running on synchronize events is harmless since --add-assignee is idempotent when the user is already assigned.

Important Files Changed

Filename Overview
.github/workflows/pr-automation.yml Renamed from run-devin.yml; adds an author-assignment step using gh pr edit with env-var-isolated inputs, placed first so browser flakiness can't block it

Reviews (2): Last reviewed commit: "Add PR author assignment and generalize ..." | Re-trigger Greptile

@andrew-polk andrew-polk marked this pull request as ready for review June 16, 2026 18:48
@JohnThomson JohnThomson deleted the branch master June 16, 2026 20:00
@andrew-polk andrew-polk reopened this Jun 16, 2026
@andrew-polk andrew-polk changed the base branch from devin to master June 16, 2026 20:41
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.

2 participants