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.

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

Add Unreviewed PRs reminder to Dev channel #31

Merged
merged 2 commits into from
Oct 11, 2022
Merged

Conversation

willm30
Copy link
Collaborator

@willm30 willm30 commented Oct 10, 2022

This PR implements an "Unreviewed PRs" reminder in the Dev discord channel. I've set the cron to tick at 8.30am London every weekday.

Chewie will fetch all PRs in colonyDapp, then filter them by prs with a "review_requested" event that is over 3 days old, but which have not yet received any reviews. If there are any, Chewie will message the Dev channel with a list of the outstanding prs:

chewie

@willm30 willm30 force-pushed the feature/github-pr-reminder branch from 40d1f26 to e4c6079 Compare October 10, 2022 08:49
@willm30 willm30 force-pushed the feature/github-pr-reminder branch from f21d8dd to 1cf0706 Compare October 10, 2022 08:54
@willm30 willm30 requested a review from area October 10, 2022 08:55
@danbr
Copy link

danbr commented Oct 10, 2022

This PR implements an "Unreviewed PRs" reminder in the Dev discord channel. I've set the cron to tick at 8.30am London every weekday.

Chewie will fetch all PRs in colonyDapp, then filter them by prs with a "review_requested" event that is over 3 days old, but which have not yet received any reviews. If there are any, Chewie will message the Dev channel with a list of the outstanding prs:

chewie

@willm30 Good initiative..... Will we been recording this stat somewhere? I guess this is how we know that we have a problem, and also the severity.

@willm30
Copy link
Collaborator Author

willm30 commented Oct 10, 2022

@danbr There's currently nothing in place to keep track of this data aside from the daily logging of outstanding PRs. Did you have something additional in mind?

@danbr
Copy link

danbr commented Oct 10, 2022

@danbr There's currently nothing in place to keep track of this data aside from the daily logging of outstanding PRs. Did you have something additional in mind?

Nothing specific, just spitballing about what might be useful in the future

@area area merged commit 311035c into master Oct 11, 2022
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.

3 participants