Skip to content

Stale label github action#5405

Open
GenevieveBuckley wants to merge 2 commits intodask:mainfrom
GenevieveBuckley:stalebot
Open

Stale label github action#5405
GenevieveBuckley wants to merge 2 commits intodask:mainfrom
GenevieveBuckley:stalebot

Conversation

@GenevieveBuckley
Copy link
Copy Markdown
Contributor

As discussed here we want a bot to automatically label stale issues in the repository, in the least obnoxious way possible.

This PR introduces a yml file for the github actions/stale, to run automatically on a regular interval.

@GenevieveBuckley
Copy link
Copy Markdown
Contributor Author

Related: dask/dask#8244

exempt-issue-labels: 'good intro to dask,good first issue,Good First Issue,good second issue,feature request'
#exempt-pr-labels: 'awaiting-approval,work-in-progress'
exempt-draft-pr: true
start-date: '2020-04-18T00:00:00Z' # ignore before this date, ISO 8601 or RFC 2822
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.

Why would we want to ignore tickets before this date? @jsignell do you know why this is in here?

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.

I guess the idea was not to apply them to all of history, but only going forward. Based on the conversation in the dev meeting, seems like we are ok with letting the bot do more.

days-before-close: -1
stale-issue-label: 'needs attention'
stale-pr-label: 'needs attention'
exempt-issue-labels: 'good intro to dask,good first issue,Good First Issue,good second issue,feature request'
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.

Suggested change
exempt-issue-labels: 'good intro to dask,good first issue,Good First Issue,good second issue,feature request'
exempt-issue-labels: 'good first issue,good second issue,feature,flaky test, enhancement'

Some of the labels in here do not exist

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.

yeah I imagine they either used to exist or they exist on dask but not distributed.

@jsignell
Copy link
Copy Markdown
Member

jsignell commented Mar 8, 2022

@GenevieveBuckley based off the conversation here: dask/community#60 (comment) we are planning on reviving this work. Is it ok with you if we push to your branch?

@GenevieveBuckley
Copy link
Copy Markdown
Contributor Author

@GenevieveBuckley based off the conversation here: dask/community#60 (comment) we are planning on reviving this work. Is it ok with you if we push to your branch?

@jsignell yes, it is always ok for you or other maintainers to push to my branches 😄 Go right ahead!

I'm sure it's also be a lot more time-efficient in this particular case, too.

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