Bienvenue, Invité
Nom d'utilisateur : Mot de passe : Se souvenir de moi

SUJET : Cloturer Les Pull Requests sur Github

Cloturer Les Pull Requests sur Github il y a 1 semaine 2 jours #115001

  • wdammak
  • Portrait de wdammak
  • Hors ligne
  • Bannis
  • Messages : 207
  • Remerciements reçus 21
  • Karma: 0
Hello,
Appel aux auteurs des PR sur github! un peu de ménage (Garbage collector!!)
Un PR qui date de 3ans ou même 2mois est obsolète et n'aura aucune chance d'être publier!
Commencez par clôturer, puis pensez à refaire les modifications sur une nouvelle version (pour éviter les conflits) et republier!

Ou bien, et à mon avis c'est plus constrictive, c'est que eldy ferait une pause et se focalise sur la validation des PR existantes, cela inciterait les autres contributeurs de proposer des nouvelles améliorations et fonctionnalités...
L'administrateur a désactivé l'accès en écriture pour le public.

Cloturer Les Pull Requests sur Github il y a 1 semaine 1 jour #115012

  • wdammak
  • Portrait de wdammak
  • Hors ligne
  • Bannis
  • Messages : 207
  • Remerciements reçus 21
  • Karma: 0
J'insiste sur le fait de ne plus laisser les PR en attente pour longtemps par eldy avant leurs publication car j'ai remarqué que beaucoup de contributeur ont délaissé le projet pour ces raisons!
par exemple @rdoursenaud était très impliqué et depuis fin 2016 a abandonné! et par hasard ça coince avec son PR qui date de 2016 pas encore publié! maintenant avec 11 fichiers en conflits c'est impossible à corriger et publier! dommage!

https://github.com/Dolibarr/dolibarr/pull/4432
Dernière édition: il y a 1 semaine 1 jour par wdammak.
L'administrateur a désactivé l'accès en écriture pour le public.

Cloturer Les Pull Requests sur Github il y a 1 semaine 1 jour #115013

  • aspangaro-OpenDSI
  • Portrait de aspangaro-OpenDSI
  • Hors ligne
  • Modérateur
  • Messages : 1355
  • Remerciements reçus 352
  • Karma: 34
Hello,

Le tri sera fait prochainement mais l'exemple au dessus est mauvais car avec un WIP dans le titre, c'est sur qu'il ne sera pas intégré. Raphaël est partie vers d'autres horizons, c'est son choix, c'est parce que un PR n'est pas intégré qu'il faut pleurer.

La meilleure solution pour faire signe et de clôturer un PR et de le recouvrir s'il date vraiment mais même moi j'en ai qui traîne et je les clôture au bout d'un certain temps pour alléger la base. Et des fois, c'est juste une question de point de vue.

Bonne soirée
L'administrateur a désactivé l'accès en écriture pour le public.

Cloturer Les Pull Requests sur Github il y a 1 semaine 1 jour #115014

  • wdammak
  • Portrait de wdammak
  • Hors ligne
  • Bannis
  • Messages : 207
  • Remerciements reçus 21
  • Karma: 0
Hello,
un autre exemple qui fait rire :laugh: https://github.com/Dolibarr/dolibarr/pull/11093
4 mois pour un "a space missing..." bon à ce rythme là ...
Heureusement que Elon Musk n'utilise pas travis dans ses entreprises car sinon spacex n'a pas vu le jour encore! :whistle:
Dernière édition: il y a 1 semaine 1 jour par wdammak.
L'administrateur a désactivé l'accès en écriture pour le public.

Cloturer Les Pull Requests sur Github il y a 1 semaine 1 jour #115015

  • pm17
  • Portrait de pm17
  • Hors ligne
  • Modérateur
  • Linux/Mac/Windows Un petit merci sur mon profil
  • Messages : 1346
  • Remerciements reçus 184
  • Karma: 23
Slt

On cherche des modos
www.dolibarr.fr/forum/12-howto--aide/649...s-traducteurs#114189

Si tu est motivé
un modo connaissant github et pouvant faire le lien avec le forum serait bien
L'administrateur a désactivé l'accès en écriture pour le public.

Cloturer Les Pull Requests sur Github il y a 1 semaine 1 jour #115016

  • Arre
  • Portrait de Arre
  • Hors ligne
  • Modérateur
  • Envie de participer ? wiki -> adhérer à l'asso :)
  • Messages : 1881
  • Remerciements reçus 428
  • Karma: 44
@pm17

vu le ton qu'emploi wdammak, ça n'est pas vraiment une bonne idée...
L'administrateur a désactivé l'accès en écriture pour le public.

Cloturer Les Pull Requests sur Github il y a 1 semaine 1 jour #115017

  • pm17
  • Portrait de pm17
  • Hors ligne
  • Modérateur
  • Linux/Mac/Windows Un petit merci sur mon profil
  • Messages : 1346
  • Remerciements reçus 184
  • Karma: 23
Slt
@Arre
Je vois pas le souci
Il a un point de vue il le defend c tout

et vu qu on a a besoin de monde et que ca se bouscule pas :whistle:
toute personne motivée est tjrs bonne a prendre
L'administrateur a désactivé l'accès en écriture pour le public.

Cloturer Les Pull Requests sur Github il y a 1 semaine 5 heures #115031

  • wdammak
  • Portrait de wdammak
  • Hors ligne
  • Bannis
  • Messages : 207
  • Remerciements reçus 21
  • Karma: 0
Autre exemple qui date depuis avril! https://github.com/Dolibarr/dolibarr/pull/10997

@Arre : yes t'as raison :)

Sinon, mon point de vue c'est que eldy peut être est débordé mais j'ai l'impression qu'il ne fait pas confiance à la communauté et à d'autres fantastiques contributeurs! (peut être je me trompe!)

Sinon j'invite les contributeurs de faire une pause et faire le ménage des anciens PR! Car j'ai vu des cas sur github où 3 dev travaillent indépendamment sur la même fonctionnalité... perte de temps et d’énergie...

Bon! as u like!
Dernière édition: il y a 1 semaine 5 heures par wdammak.
L'administrateur a désactivé l'accès en écriture pour le public.

Cloturer Les Pull Requests sur Github il y a 6 jours 15 heures #115051

  • jtraulle
  • Portrait de jtraulle
  • Hors ligne
  • Admin
  • Messages : 226
  • Remerciements reçus 83
  • Karma: 15
@wdammak : Les critiques constructives sont prises en compte et appréciées. J'ai par exemple mis en place avec Eldy SticklerCI pour avoir une vérification du code style plus simple et des résolutions automatiques lorsque c'est possible. De façon plus importante, le fait d'utiliser Stickler CI fait que les vérifications sur le Code Style sont maintenant effectuées uniquement sur les fichiers modifiés de la PR (et non plus sur tous les fichiers de la branche ce qui était le cas auparavent). Cette étape ayant été supprimée de TravisCI vu qu'elle est maintenant menée par SticklerCI permet de gagner du temps d'exécution sur le TravisCI.

wdammak écrit:
Sinon, mon point de vue c'est que eldy peut être est débordé mais j'ai l'impression qu'il ne fait pas confiance à la communauté et à d'autres fantastiques contributeurs! (peut être je me trompe!)
Personnellement, je ne le perçois pas comme ça mais ce n'est pas toujours évident de savoir à qui on peut se fier. La confiance est une chose fragile qui se gagne progressivement. Et, oui, c'est certain que Eldy a beaucoup de choses à gérer.

wdammak écrit:
Sinon j'invite les contributeurs de faire une pause et faire le ménage des anciens PR! Car j'ai vu des cas sur github où 3 dev travaillent indépendamment sur la même fonctionnalité... perte de temps et d’énergie...
Dans ces cas là, peux tu ajouter un commentaire sur les PR GitHub concernées en liant les autres PR en ajoutant leur numéro précédé d'un # ? Cela aidera tout le monde et permettra au moins aux différents dév de savoir qu'il y a peut-être un travail en cours en double (ou triple).

Le triage des issues et des PR va être amélioré. C'est un processus en cours et il faut garder à l'esprit que l'ensemble des contributeurs ont des occupations autre que Dolibarr ;) Je vais par exemple essayer de voir pour fermer les très anciennes PR (on est d'accord que garder des reliques de cet ancien temps ouvertes n'a pas beaucoup de sens ... surtout quand plusieurs versions majeures ont été publiées depuis).

Je tiens cependant à dire que pour avoir moi même proposé plusieurs PR, elles ont toujours été traitées relativement rapidement. A partir du moment où le développeur soumettant la PR est présent, accepte la critique constructive et réponds ou corrige les éventuelles choses remontées par la personne effectuant la revue de code (Eldy), les choses sont assez fluides et fonctionnent bien.
L'administrateur a désactivé l'accès en écriture pour le public.

Cloturer Les Pull Requests sur Github il y a 5 jours 14 heures #115084

  • pm17
  • Portrait de pm17
  • Hors ligne
  • Modérateur
  • Linux/Mac/Windows Un petit merci sur mon profil
  • Messages : 1346
  • Remerciements reçus 184
  • Karma: 23
Slt a tous

jtraulle écrit:
Le triage des issues et des PR va être amélioré. C'est un processus en cours et il faut garder à l'esprit que l'ensemble des contributeurs ont des occupations autre que Dolibarr ;) Je vais par exemple essayer de voir pour fermer les très anciennes PR (on est d'accord que garder des reliques de cet ancien temps ouvertes n'a pas beaucoup de sens ... surtout quand plusieurs versions majeures ont été publiées depuis).

@jtraulle
Je propose que @wdammak fasse ici de temps une liste des PR a probleme
afin que tu jette un oeil
Serait tu ok ?
Dernière édition: il y a 5 jours 14 heures par pm17.
L'administrateur a désactivé l'accès en écriture pour le public.

Cloturer Les Pull Requests sur Github il y a 5 jours 13 heures #115085

  • jtraulle
  • Portrait de jtraulle
  • Hors ligne
  • Admin
  • Messages : 226
  • Remerciements reçus 83
  • Karma: 15
Le plus simple, vu que @wdammak a déjà un compte sur GitHub, c'est encore qu'il commente sur les PR qu'il juge problématiques et/ou obsolète (il peut me mentionner sur GitHub (@jtraulle) si besoin de fermer une PR également) ;)

Comme cela, ça évitera les allez retour.
L'administrateur a désactivé l'accès en écriture pour le public.

Cloturer Les Pull Requests sur Github il y a 5 jours 11 heures #115094

  • wdammak
  • Portrait de wdammak
  • Hors ligne
  • Bannis
  • Messages : 207
  • Remerciements reçus 21
  • Karma: 0
Dernière édition: il y a 5 jours 11 heures par wdammak.
L'administrateur a désactivé l'accès en écriture pour le public.

Cloturer Les Pull Requests sur Github il y a 5 jours 7 heures #115106

  • jtraulle
  • Portrait de jtraulle
  • Hors ligne
  • Admin
  • Messages : 226
  • Remerciements reçus 83
  • Karma: 15
Merci, j'ai passé en revue cette liste et réassigné les PR aux bonnes personnes ;)
L'administrateur a désactivé l'accès en écriture pour le public.
Cet utilisateur a été remercié pour son message par: wdammak, dolibarr95

Cloturer Les Pull Requests sur Github il y a 5 jours 7 heures #115107

  • ptibogxiv
  • Portrait de ptibogxiv
  • Hors ligne
  • Gold Boarder
  • Messages : 399
  • Remerciements reçus 80
  • Karma: 12
fait pour ma part ;)
L'administrateur a désactivé l'accès en écriture pour le public.
Cet utilisateur a été remercié pour son message par: wdammak

Cloturer Les Pull Requests sur Github il y a 4 jours 12 heures #115125

  • wdammak
  • Portrait de wdammak
  • Hors ligne
  • Bannis
  • Messages : 207
  • Remerciements reçus 21
  • Karma: 0
Merci à vous deux
@jtraulle je suggère d'ajouter d'autres tags/label comme:
Improvement
TBS (To Be Specified)
To Do
waiting for author (Status: Waiting for Author Feedback)
waiting for dev (Status: Waiting for dev Feedback)
CO (Core)
Performance
Regression
Modules
Themes
Products
Invoices
Order
Stock
Inventory
Accounting
Multi company
MBR
Combinations
website
Translations
Import
Export
...
L'administrateur a désactivé l'accès en écriture pour le public.