Nettoyage du repository Github Gladys 🧹

Salut Ă  tous @contributors !

J’ai pris un peu de temps pour fermer pas mal de PR sur le repo Github Gladys :slight_smile:

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 :slight_smile:

Si par erreur, j’ai fermée une PR qui ne devait pas être fermée, n’hésiter pas à la rouvrir !

2 Likes

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 Like

ç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.

On peut mettre en place un bot pour ça fermer les PR qui n’ont pas bougé depuis 30 / 60 jours , non ?

1 Like

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

?

Oui on peut avec le stale-bot

1 Like

@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 :slight_smile:

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.

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 Like

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 :slight_smile:

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 :slight_smile:

Mon compte Gmail est on :fire: :sweat_smile:

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 Like