Salut Ă tous @contributors !
J’ai pris un peu de temps pour fermer pas mal de PR sur le repo Github Gladys
Une grande partie de ces PRs étaient complètement inactive depuis plus de 6 mois, avec “WIP” dans le titre, et la plupart de temps les tests cassés.
De mon côté, c’était de plus en plus compliqué de m’y retrouver sur Github, je trouve jamais les PRs où on me mentionne, et ça fait perdre un temps fou à tout le monde.
En plus de ça, je pense que ça doit décourager les nouveaux développeurs qui arrivent sur le projet et qui voient qu’il y a une centaine de PR non mergée dont la plupart sont totalement cassée, ça donne l’impression que le projet est à l’abandon, ce qui n’est pas du tout le cas
Si par erreur, j’ai fermée une PR qui ne devait pas être fermée, n’hésiter pas à la rouvrir !
2 « J'aime »
Pour info, de mon côté, les PRs assez vieilles sur lesquelles je n’ai pas terminé de travailler, je les ai simplement recréée sur mon fork personnel afin de ne pas les perdre.
1 « J'aime »
ça marche! Effectivement c’est peut-être plus simple de les garder hors du repo principal tant qu’elles ne sont plus en développements actifs.
VonOx
Octobre 15, 2021, 7:06
4
On peut mettre en place un bot pour ça fermer les PR qui n’ont pas bougé depuis 30 / 60 jours , non ?
1 « J'aime »
Si on peut le faire en 2 coups.
Au bout de 30 jours, 1er message “attention on va la fermer automatiquement”
Au bout de 60 jours, fermeture
?
VonOx
Octobre 17, 2021, 10:48
6
Oui on peut avec le stale-bot
1 « J'aime »
@VonOx Yes je suis d’accord, je vais regarder ça !
Hop, ajout du stale-bot avec la configuration suivante:
J’ai ajouté le bot sur les deux repos: Gladys et v4-website
A voir à l’usage, on peut modifier la configuration si ça nous parait trop/pas assez agressif.
Dans tous les cas, ça va grandement nous donner un coup de main pour fermer les issues/PR qui sont inactives, et qui:
Donnent l’impression a un potentiel contributeur que le projet est abandonné, alors que ce n’est pas le cas (la plupart des PRs sont juste sans réponses/plus d’actualités…)
Permettre aux mainteneurs de mieux cerner les PRs vraiment active, et ainsi de ne pas se perdre dans une liste infinie de PR/d’issue comme c’est actuellement.
En revanche, on ne veut pas:
Perdre des bugfix remontés et qu’on a pas pu fixer en 2 mois ( 2 mois, c’est très très court pour fixer une issue, et en tout cas personnellement j’ai rarement le temps de m’attaquer une issue en 2 mois sauf si c’est critique/un tout petit dev qui peut se glisser en priorité ).
Perdre des développements poussés qui était juste en stand-bye.
Hésitez pas si ça ferme des PRs/Issues importantes à me faire un retour, la configuration est modifiable au besoin.
VonOx
Octobre 18, 2021, 5:54
9
Le truc c’est qu’il faut ignorer les issues, j’ai tenter cette action
J’ai pas mis de filtre sur les labels mais ça donne une idée
Je suis pas d’accord, pourquoi vouloir ignorer les issues ? On a le même problème de pollution sur les issues, si ce n’est plus.
Pour les issues, je suis du meme avis que @VonOx .
Si un nouveau créé une issue et que personne répond, elle sera fermée.
Du coup on ne traitera jamais le problème.
1 « J'aime »
Justement, si il n’y a plus de réponses pendant 2 mois, ça fait sens pour moi d’être alerté par le bot « qu’attention ça va peut-être être fermé » (ce qui fait une notification aux gens concernés) :
Si ça ne doit pas être fermé, on peut soit répondre, soit même un tag spécifique pour préciser que le bug est bien toujours présent et qu’on va le traiter.
Si ça doit être fermé, on touche à rien et le bot fermera l’issue
Dans tous les cas, l’objectif ici n’est pas de perdre des retours de bugs (on veut clairement éviter ça), c’est d’avoir un petit rappel du bot pour dire « attention les gars, 2 mois que personne répond ici, est-ce que c’est toujours d’actualité ? »
Parce que clairement il y a pas mal d’issues qui sont zombies dans le projet, et je pense qu’on manque d’une hygiène de clean automatique
La première vague de messages du bot est passée!
Comme il y a du rate-limite côté Github, il ne fait que répond à 30 issues par passe au maximum, donc ça va passer sur tout le repo en quelques passes.
Je vais regarder chaque issue pour ce premier check pour vérifier si il n’y a pas eu d’erreur
Bon c’est vraiment génial stale-bot, je suis 100% d’accord avec tout ce qui nous sort.
Je me fais un par un chaque notification, et soit:
C’est un truc qui doit être sur le forum, et je créé le post sur le forum et ferme l’issue github
C’est vieux et je ferme
C’était pas moi qui avait créé l’issue, auquel cas le participant à 7 jours pour se manifester.
Dans tous les cas, ça fait bouger les choses et c’est positif
lol c’est violent !
spam de notif github + mails
mais ça semble déjà porter ses fruits.
1 « J'aime »