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

Add stale action for PR and Issues #2383

Merged
merged 3 commits into from
Oct 17, 2023

Conversation

erickisos
Copy link
Contributor

@erickisos erickisos commented Oct 9, 2023

Esta es una propuesta de reemplazo a ladibug para gestionar el estado de Issues y PRs que han estado sin movimiento en un periodo de tiempo.

Closes #628
Reemplaza #629

Copy link
Collaborator

@cmaureir cmaureir left a comment

Choose a reason for hiding this comment

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

un par de comentarios en el texto. Estaría bien probarlo :)

.github/workflows/stale.yaml Outdated Show resolved Hide resolved
.github/workflows/stale.yaml Outdated Show resolved Hide resolved
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
@erickisos erickisos requested a review from cmaureir October 16, 2023 16:28
Copy link
Collaborator

@cacrespo cacrespo left a comment

Choose a reason for hiding this comment

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

Está muy bueno esto.
Si no entendí mal, quedaría del siguiente modo:

  • A los 10 días de que un issue no tenga actividad se taggea y se escribe el mensaje.
  • 7 días para PR
  • y 21 días se cierra el PR

¿es así? +1 para probarlo!

.github/workflows/stale.yaml Outdated Show resolved Hide resolved
name: 'Cierra issues y PRs antiguos'
on:
schedule:
- cron: '30 1 * * *'
Copy link
Collaborator

Choose a reason for hiding this comment

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

¿A esa hora es por algo en particular? Me da curiosidad jej...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

número random jajaja, podemos cambiarla pues

Co-authored-by: Carlos A. Crespo <lvccrespo@gmail.com>
@erickisos
Copy link
Contributor Author

Justo @cacrespo esa es básicamente la configuración, igual puedo agregar la doc del Github Action por acá y vamos jugando a ver qué tal nos funciona

Copy link
Collaborator

@cmaureir cmaureir left a comment

Choose a reason for hiding this comment

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

Yo creo que lo podemos probar. En el peor de los casos siempre lo podemos quitar :)

@cmaureir cmaureir merged commit 169b71e into python:3.11 Oct 17, 2023
1 check passed
@rtobar
Copy link
Collaborator

rtobar commented Oct 17, 2023

Esto quedó mergeado en 3.11 😢

@erickisos erickisos deleted the issue/628-stale-action branch October 18, 2023 19:49
@cmaureir
Copy link
Collaborator

for the record, ya se hizo el merge en 3.12

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.

automatizar "pings" en issues/prs sin novedades
4 participants