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 4 jours 12 heures #115127

  • dolibarr95
  • Portrait de dolibarr95
  • Hors ligne
  • Admin
  • Messages : 1270
  • Remerciements reçus 188
  • Karma: 25
Bonjour :)
A mon avis trop de labels ça embrouille un peu
L'administrateur a désactivé l'accès en écriture pour le public.

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

  • 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
dolibarr95 écrit:
Bonjour :)
A mon avis trop de labels ça embrouille un peu
+1

Pas d interet de label pour savoir a quel module correspond le code
ya le titre pour ca non ?

mais c que mon avis
propose ton idee a @eldy si tu pense que c est utile
L'administrateur a désactivé l'accès en écriture pour le public.
Cet utilisateur a été remercié pour son message par: dolibarr95

Cloturer Les Pull Requests sur Github il y a 4 jours 11 heures #115135

  • jtraulle
  • Portrait de jtraulle
  • Hors ligne
  • Admin
  • Messages : 226
  • Remerciements reçus 83
  • Karma: 15
J'avais demandé à Eldy pour trier par module mais pour lui, ça ne sert à rien car ce regroupement n'apporterait aucune plus value.
Pour le reste, les suggestions de tags doivent être vues directement avec Eldy (je peux juste trier = ajouter, enlever des tags sur les issues et PR, assigner des personnes mais pas gérer les tags en eux même = en ajouter/modifier/supprimer).
Dernière édition: il y a 4 jours 11 heures par jtraulle.
L'administrateur a désactivé l'accès en écriture pour le public.

Cloturer Les Pull Requests sur Github il y a 4 jours 9 heures #115142

  • wdammak
  • Portrait de wdammak
  • Hors ligne
  • Bannis
  • Messages : 207
  • Remerciements reçus 21
  • Karma: 0
Pour un dev avoir toutes les PR en relation avec Accounting ça lui permet d'avoir une idée sur le stade des dev en cours par un simple clique sur le label et ça lui évite de travailler et proposer la même chose

Un ERP est un pack très vaste de multi-applications/modules, le triage est important pour plusieurs besoins... je peux être intéressé par la gestion de stock mais jamais de la comptabilité ou la gestion des comptes bancaires...

Parcourir toutes les pages et lire toutes les titres pour comprendre de quoi il s'agit c'est fatiguant!

Pour l'administrateur du projet ça lui permet d'organiser les priorités...
...
Mais bon! à vous de voir!
L'administrateur a désactivé l'accès en écriture pour le public.

Cloturer Les Pull Requests sur Github il y a 4 jours 8 heures #115145

  • 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
Avec environ 80 modules natifs
ca va faire une sacré liste :silly:
A mon avis si tu veux "vendre" cette idee a eldy
il vaudrait mieux se limiter aux "familles"
Ex Produit/service/stock....
L'administrateur a désactivé l'accès en écriture pour le public.

Cloturer Les Pull Requests sur Github il y a 4 jours 8 heures #115147

  • wdammak
  • Portrait de wdammak
  • Hors ligne
  • Bannis
  • Messages : 207
  • Remerciements reçus 21
  • Karma: 0
Je vois aucun problème d'avoir 300 label! ils ne seronts visible que quand on l'assigne à un PR
L'administrateur a désactivé l'accès en écriture pour le public.

Cloturer Les Pull Requests sur Github il y a 3 jours 20 heures #115162

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

Ou plutôt que de rajouter des tags pour chaque module, préciser à quel module cela correspond dans le titre du PR, en plus c'est mieux pour le changelog.

J'essaye de mettre Accountancy / Salaries / Loan / etc...

Au moins c'est clair comme ça :P
L'administrateur a désactivé l'accès en écriture pour le public.

Cloturer Les Pull Requests sur Github il y a 3 jours 19 heures #115165

  • dolibarr95
  • Portrait de dolibarr95
  • Hors ligne
  • Admin
  • Messages : 1270
  • Remerciements reçus 188
  • Karma: 25
Bonjour :)
ah ben tiens voila un truc qui me tracasse...comment le changelog est-il alimenté : chaque pr ajoute une ligne ?
L'administrateur a désactivé l'accès en écriture pour le public.

Cloturer Les Pull Requests sur Github il y a 3 jours 19 heures #115167

  • jtraulle
  • Portrait de jtraulle
  • Hors ligne
  • Admin
  • Messages : 226
  • Remerciements reçus 83
  • Karma: 15
Chaque commit qui commence par NEW ou FIX en majuscules oui.
L'administrateur a désactivé l'accès en écriture pour le public.
Cet utilisateur a été remercié pour son message par: dolibarr95

Cloturer Les Pull Requests sur Github il y a 3 jours 19 heures #115172

  • dolibarr95
  • Portrait de dolibarr95
  • Hors ligne
  • Admin
  • Messages : 1270
  • Remerciements reçus 188
  • Karma: 25
Merci ;)

Mais si je prend une pr au hasard qui commence par "FIX:"
github.com/Dolibarr/dolibarr/pull/11368
je ne vois pas la ligne correspondante dans le changelog ?
raw.githubusercontent.com/Dolibarr/dolibarr/develop/ChangeLog
L'administrateur a désactivé l'accès en écriture pour le public.

Cloturer Les Pull Requests sur Github il y a 3 jours 19 heures #115173

  • jtraulle
  • Portrait de jtraulle
  • Hors ligne
  • Admin
  • Messages : 226
  • Remerciements reçus 83
  • Karma: 15
dolibarr95 écrit:
Merci ;)

Mais si je prend une pr au hasard qui commence par "FIX:"
github.com/Dolibarr/dolibarr/pull/11368
je ne vois pas la ligne correspondante dans le changelog ?
raw.githubusercontent.com/Dolibarr/dolibarr/develop/ChangeLog

Cette pr est pour la version 8 ;)
L'administrateur a désactivé l'accès en écriture pour le public.

Cloturer Les Pull Requests sur Github il y a 3 jours 19 heures #115175

  • dolibarr95
  • Portrait de dolibarr95
  • Hors ligne
  • Admin
  • Messages : 1270
  • Remerciements reçus 188
  • Karma: 25
Ok, mais cette ligne ne devrait-elle pas apparaître sous :
***** ChangeLog for 8.0.0 compared to 7.0.0 *****
L'administrateur a désactivé l'accès en écriture pour le public.

Cloturer Les Pull Requests sur Github il y a 3 jours 19 heures #115177

  • jtraulle
  • Portrait de jtraulle
  • Hors ligne
  • Admin
  • Messages : 226
  • Remerciements reçus 83
  • Karma: 15
L'administrateur a désactivé l'accès en écriture pour le public.

Cloturer Les Pull Requests sur Github il y a 3 jours 19 heures #115179

  • dolibarr95
  • Portrait de dolibarr95
  • Hors ligne
  • Admin
  • Messages : 1270
  • Remerciements reçus 188
  • Karma: 25
:whistle: ok je sors....
L'administrateur a désactivé l'accès en écriture pour le public.

Cloturer Les Pull Requests sur Github il y a 2 jours 12 heures #115282

  • wdammak
  • Portrait de wdammak
  • Hors ligne
  • Bannis
  • Messages : 207
  • Remerciements reçus 21
  • Karma: 0
https://github.com/Dolibarr/dolibarr/commits/develop

En 4 ou 5 jours presque aucun PR des contributeurs a été validé! à ce rythme là... :whistle:
Dernière édition: il y a 2 jours 12 heures par wdammak.
L'administrateur a désactivé l'accès en écriture pour le public.