Skip to content

Update bot to close PRs waiting on authors for more than 3 months - #8735

Merged
sklam merged 3 commits into
numba:mainfrom
guilhermeleobas:guilhermeleobas/stale-prs
Feb 15, 2023
Merged

sklam merged 3 commits into
numba:mainfrom
guilhermeleobas:guilhermeleobas/stale-prs

Conversation

@guilhermeleobas

Copy link
Copy Markdown
Contributor

Following some recent discussion on Numba public meeting. It was decided that we will close PRs waiting for author response for more than 3 months. Since we already had a bot for staling PRs, I just updated it to accommodate the new changes.

@stuartarchibald stuartarchibald left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for making these changes @guilhermeleobas, couple of queries to resolve else looks good.

Comment thread .github/workflows/stale.yml Outdated
Comment on lines +30 to +34
This pull request is marked as stale as it has had no activity in
the past 6 months. Please respond to this comment if you're still
interested in working on this.
days-before-pr-stale: 90 # 3 months
days-before-pr-close: 90

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The message text refers to 6 months, but the days before for stale is 3 months (90 days)? Also, is there an option to set a message for when it's closed?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

Also, is there an option to set a message for when it's closed?

This is the message that will show up. See https://github.com/actions/stale#stale-pr-message

Comment thread .github/workflows/stale.yml Outdated
days-before-pr-stale: 90 # 3 months
days-before-pr-close: 90
any-of-pr-labels: '2 - In progress,4 - Waiting on author'
stale-pr-label: 'abandoned'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When is this label applied? At the point it is considered "stale"? If so, should it be labelled "stale" at this point and then "abandoned" once it is actually closed?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

@stuartarchibald stuartarchibald added 4 - Waiting on author Waiting for author to respond to review and removed 3 - Ready for Review labels Feb 6, 2023
@stuartarchibald stuartarchibald self-assigned this Feb 6, 2023
@guilhermeleobas guilhermeleobas added 4 - Waiting on reviewer Waiting for reviewer to respond to author and removed 4 - Waiting on author Waiting for author to respond to review labels Feb 6, 2023

@stuartarchibald stuartarchibald left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fixes @guilhermeleobas, I think this achieves what was agreed by the FPOC group a few weeks back but someone else ought to take a look too.

Comment thread .github/workflows/stale.yml Outdated
@stuartarchibald stuartarchibald added the 4 - Waiting on second reviewer Patch needs a second reviewer. label Feb 15, 2023
@stuartarchibald stuartarchibald added this to the Numba 0.57 RC milestone Feb 15, 2023
Co-authored-by: stuartarchibald <stuartarchibald@users.noreply.github.com>

@sklam sklam left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sklam sklam added 5 - Ready to merge Review and testing done, is ready to merge and removed 4 - Waiting on reviewer Waiting for reviewer to respond to author 4 - Waiting on second reviewer Patch needs a second reviewer. labels Feb 15, 2023
@stuartarchibald stuartarchibald added the Effort - short Short size effort needed label Feb 15, 2023
@sklam
sklam merged commit 7d0e129 into numba:main Feb 15, 2023
@guilhermeleobas
guilhermeleobas deleted the guilhermeleobas/stale-prs branch February 16, 2023 00:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

5 - Ready to merge Review and testing done, is ready to merge Effort - short Short size effort needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants