Skip to content

Conversation

adegeo
Copy link
Contributor

@adegeo adegeo commented Jun 2, 2022

Summary

I looked into the no response bot and it seems it's been deprecated for a year. They suggested using this action.

@gewarren

Do we need to add auth for this action in the github settings?

How do we remove the no-response bot app?

@adegeo adegeo requested a review from a team as a code owner June 2, 2022 16:31
@dotnet-bot dotnet-bot added this to the June 2022 milestone Jun 2, 2022
Copy link
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

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

This LGTM @adegeo

My only suggestion is to run it once a day, instead of every hour.

@gewarren
Copy link
Contributor

gewarren commented Jun 3, 2022

It's too bad that the action in lives in a private organization. Is there not something in the dotnet org we can use?

@adegeo
Copy link
Contributor Author

adegeo commented Jun 3, 2022

@gewarren We can fork it and reference it?

Co-authored-by: Bill Wagner <wiwagn@microsoft.com>
@IEvangelist
Copy link
Member

IEvangelist commented Jun 10, 2022

Hey @adegeo, @BillWagner, and @gewarren

I'm curious, how exactly does this work? If someone posts an issue, and we don't touch it for 14 days - does it get closed? Ideally, that wouldn't be the case. How does it know when we engage, is that the issue_comment part of the trigger? This seems a bit incorrect as is, as it would have no way of knowing if a comment was from a bot and not one of the team members.

Co-authored-by: Youssef Victor <youssefvictor00@gmail.com>
@adegeo adegeo merged commit 75a9973 into main Jun 15, 2022
@adegeo adegeo deleted the adegeo-noresponse branch June 15, 2022 15:34
@adegeo
Copy link
Contributor Author

adegeo commented Jun 15, 2022

@IEvangelist The issue comment trigger is only used for detecting when the issue author actually responds after the "needs more info" label was added. The label is removed and if the issue was closed, it's reopened.

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.

6 participants