Skip to content

Conversation

@PLeVasseur
Copy link
Collaborator

Summary

  • track review completion on PR approvals and issue sign-off label
  • update reviewer activity on comment or changes requested
  • skip completed reviews in overdue checks
  • wire pull_request_review event context into the workflow

Testing

  • uv run pytest .github/reviewer-bot-tests
  • uv run ruff check --fix

Notes

  • Documentation updates will be moved to PR 331

Closes #388

@netlify
Copy link

netlify bot commented Feb 4, 2026

Deploy Preview for scrc-coding-guidelines ready!

Name Link
🔨 Latest commit 952f2a9
🔍 Latest deploy log https://app.netlify.com/projects/scrc-coding-guidelines/deploys/6983a303a6b8040008caf50a
😎 Deploy Preview https://deploy-preview-391--scrc-coding-guidelines.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@PLeVasseur PLeVasseur changed the title fix(reviewer-bot): stop reminders after approvals Fixing various things, including the PR review not being accepted Feb 4, 2026
@PLeVasseur PLeVasseur merged commit 9ecc439 into rustfoundation:main Feb 4, 2026
16 checks passed
@PLeVasseur PLeVasseur deleted the fix/issue-388-review-approval branch February 4, 2026 19:52
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.

Fix bug preventing PR reviewers from giving explicit sign-off that they approve the PR to the guidelines-bot

1 participant