-
Notifications
You must be signed in to change notification settings - Fork 292
feat(stale): also close discussions #4620
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
Conversation
This PR has been open for 7 days with no objections. Just confirming with @nodejs/actions this external action is okay? |
cc @nodejs/security-wg |
it doesnt seems very popular https://github.com/steffen-karlsson/stalesweeper 😐 I'd rather use something maintained and with a community |
I would as well, but I literally could not find a single other action that did this. |
Confirming the best course of action here? 👎 No stale bot 👍 Use the 3rd-party stale bot. I've done a look through of the stale bot and I don't see any issues security-wise. Plus, since it's SHA pinned, we can be sure the content matches our expectations. ❤️ Create a 1st-party stale bot for discussions (like a script for this workflow to call, stored under For any action, in a worst case scenario, the impact is minimal: the worst thing a potential attacker would be able to do would be editing issues/discussions/PRs in this repository. |
cc: @nodejs/issue-triage |
Bump on my previous question |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ffj
Closing due to inactivity. |
Now that this repository is going to be hosting the Node.js discussions, I think it'd be nice to have a stalebot to regulate said discussions.