Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update the PR Review Reminder DAG with special timing for non-maintainers #4327

Closed
zackkrida opened this issue May 14, 2024 · 0 comments · Fixed by #4385
Closed

Update the PR Review Reminder DAG with special timing for non-maintainers #4327

zackkrida opened this issue May 14, 2024 · 0 comments · Fixed by #4385
Assignees
@AetherUnbound
Labels
🤖 aspect: dx Concerns developers' experience with the codebase 🧰 goal: internal improvement Improvement that benefits maintainers, not users 🟨 priority: medium Not blocking but should be addressed soon 🧱 stack: mgmt Related to repo management and automations

Comments

@zackkrida
Copy link
Member

Problem

As identified in the Maintainer documentation here, we would like to keep the maintainer response time for reviewing community PRs as quick as possible. 3 days is expected to be reasonable.

Description

Update the PR Review Reminder DAG to check if the PR author is not a maintainer and set the urgency to 3 days.
@zackkrida zackkrida added 🟨 priority: medium Not blocking but should be addressed soon 🤖 aspect: dx Concerns developers' experience with the codebase 🧰 goal: internal improvement Improvement that benefits maintainers, not users 🧱 stack: mgmt Related to repo management and automations labels May 14, 2024
@zackkrida zackkrida changed the title Update the PR Review Reminder DAG with an exception for non-maintainers Update the PR Review Reminder DAG with special timing for non-maintainers May 14, 2024
@AetherUnbound AetherUnbound self-assigned this May 23, 2024
@AetherUnbound AetherUnbound mentioned this issue May 24, 2024
Merged
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AetherUnbound AetherUnbound

Labels
🤖 aspect: dx Concerns developers' experience with the codebase 🧰 goal: internal improvement Improvement that benefits maintainers, not users 🟨 priority: medium Not blocking but should be addressed soon 🧱 stack: mgmt Related to repo management and automations
Projects
Openverse Backlog
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Always assume special urgency for contributor PR pings WordPress/openverse
2 participants
@zackkrida @AetherUnbound