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

Revert "[workflow] Add workflows/stale.yml to detect stale issues and PRs" #1035

Merged
merged 1 commit into from
May 22, 2020

Conversation

yuanming-hu
Copy link
Member

@yuanming-hu yuanming-hu commented May 22, 2020

Reverts #1020

Reason: #1020 (comment)

TBH I don't think people would like to have their mailboxes filled with information like these...

Screenshot from 2020-05-21 20-27-14

Even after most issues are already marked stale, I don't think people really need an email notification, since they already gave up on the issue anyway.

@yuanming-hu yuanming-hu requested review from archibate and k-ye May 22, 2020 00:27
@archibate
Copy link
Collaborator

Thank for fixing this, my mailbox is filled with action-bot now, it would be great if stale could only make label instead of leaving a comment...

@yuanming-hu
Copy link
Member Author

Thank for fixing this, my mailbox is filled with action-bot now, it would be great if stale could only make label instead of leaving a comment...

Thanks for understanding. Please always keep in mind that conveying information effectively does not mean sending out tons of messages that people can't process. I noticed that a few people have already unwatched Taichi...

@yuanming-hu yuanming-hu merged commit 28ae40c into master May 22, 2020
@k-ye
Copy link
Member

k-ye commented May 22, 2020

Ah, sorry for whoever is being affected! We should definitely be more careful about the bots..

@archibate archibate deleted the revert-1020-archibate-patch-5 branch June 17, 2020 06:30
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