Cloturer Les Pull Requests sur Github

Je vois aucun problème d’avoir 300 label! ils ne seronts visible que quand on l’assigne à un PR

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

Bonjour :happy:
ah ben tiens voila un truc qui me tracasse…comment le changelog est-il alimenté : chaque pr ajoute une ligne ?

Chaque commit qui commence par NEW ou FIX en majuscules oui.

1 « J'aime »

Merci :wink:

Mais si je prend une pr au hasard qui commence par « FIX: »


je ne vois pas la ligne correspondante dans le changelog ?
https://raw.githubusercontent.com/Dolibarr/dolibarr/develop/ChangeLog

Cette pr est pour la version 8 :wink:

Ok, mais cette ligne ne devrait-elle pas apparaître sous :
***** ChangeLog for 8.0.0 compared to 7.0.0 *****

:whistle: ok je sors…

https://github.com/Dolibarr/dolibarr/commits/develop

En 4 ou 5 jours presque aucun PR des contributeurs a été validé! à ce rythme là… :whistle:

Et c’est pas parce que tu râleras que ça ira plus vite. Dolibarr, c’est pas une multinationale :lol: :silly:

De patience jeune padawan tu t armeras!

1 « J'aime »

Je réplique?
ça se voit
D’abord, as-tu déjà travaillé dans une multinationale fondée autour d’un projet open source ?
(Bon! pls on passe!)

@wdammak

wikipedia ? la plus part des call center ? la plus part des hébergeurs comme OVH ? les éditeurs de logiciels/distributions linux : canonical… red hat… IBM …)
Sans compter tous ceux qui utilisent de l’opensource, sans même le savoir (les CMS par exemple)

bref…

va respirer loin du forum, ça te calmera les nerfs.
quand tu reviendra, soit actif et constructif.

Les remarques acerbes, sur ce post et sur d’autres, ne servent à rien.

compte suspendu 3 jours.

1 « J'aime »

Pour rappel et pour comprendre les règles de fonctionnement de la communauté Dolibarr sur github, c’est ici : https://github.com/Dolibarr/dolibarr/blob/develop/.github/CONTRIBUTING.md

Donc oui, pas de merge pendant 4-5 jours c’est bien possible.
Donc oui 100+ PR ouvertes c’est bien possible, surtout quand 95% ont été vues, taguées ou ont eu un commentaire.
Donc oui, il se peut qu’une PR passe à la trappe.
Et avec tout ça on a de + en + de monde dans cette communauté.

1 « J'aime »

Bonjour à tous,
Une petite réflexion de ma part à ce sujet, de moi tout petit dev :blush:
L’ensemble du fonctionnement est structuré et c’est plutôt rassurant non ? Effectivement cela n’avance pas toujours aussi si vite que chacun le souhaiterait mais ça avance quand même. Il faudrait un peu plus de monde à la validation des PR mais quels profils pour soulager Eldy ?

@+

Le droit de réponse est garanti! nespas? :happy:
@Arre Visiblement vous avez mal compris mon post! car je fais partir des personnes qu’y ont fait leur carrières au tour de l’open source mais différemment à dolibarr! Mon premier et unique boulot était en open source dans un appart près de la défense avec seulement 6 collègues et je l’ai quitté, avec ses vieux cadres, après 6ans(les américains* ont débarqué et voulait acheter la société pour avoir le contrôle des décisions* ) en quittant plus de 6 milles collègues dans les trois continents avec des options et des actions dans l’open source … Donc je ne suis pas entrain de se moquer de l’open source mais j’ai cru que jtraulle a lut mes premiers posts sur le forum et a cherché la provocation! (Et pourtant ma réponse était poli et j’ai rien contre lui! ni contre vous d’ailleurs!)

En tout cas, revenant au sujet, il y a un problème dans la gestion des PR et je ne suis pas le seul qui l’a fait remarquer ni du genre qui y a sa langue dans sa poche! :happy:

* heureusement ils ont fini par la faire entrer à nasdaq, ils sont des experts dans la gestion des projets open sources…

Maintenant tu pourrais me re-suspendre! :happy:

Salut wdammak :happy:

Re-bienvenue parmi nous!
Tu n’as apparemment rien contre dolibarr ou des personnes sur ce forum, mais des rancœurs, que tu expliques très bien, contre les « conflits d’intérêts » entre les initiatives de sociétés privées et l’esprit open source. Et c’est tout à ton honneur pour la droiture d’esprit.

L’écosystème de dolibarr repose sur une association, un noyau dur de développeur, sur une communauté, des développeurs indépendants, des intégrateurs, des hébergeurs, des utilisateurs…
C’est dans ce sens, je pense, que la notion de « Dolibarr, c’est pas une multinationale » a été abordée: l’organisation de tout ce petit monde est complexe :wink:

Cependant ce que tu as laissé transparaître sur le forum étaient des critiques acerbes , parfois (pas toujours) vides de proposition et limite « agressives » : en tout cas, elles ont été interprétées ainsi, et pas que par moi.

Bref, tout est affaire de forme en fait. (sans refuser la critique bien sûr, au contraire)

Tu as tout loisir de continuer à participer au forum, voir t’investir dans différents projets si tu en as le temps :happy:

Sans rancune ?

1 « J'aime »

@Wdammak : Bienvenue dans le monde des grandes gueules censurées :happy:
c’est vrai que c’est pas parce qu’on râle que ça va plus vite, mais c’est aussi vrai que c’est pas parce qu’on remonte les problèmes ou on les « FIX » qu’ils sont résolus ou en cours de résolution :wink:
Combien de PR ont été « closed » parce que pas « merged » en temps opportun ?
Loin de moi l’idée de jeter la pierre à Eldy ou tous les Yodas qui savent relire et valider/invalider des PR, mais si un ordre de passage / des priorités n’est pas défini et respecté (comme ce qui est en devenir actuellement : PR postponed / PR en discussion…) il y aura toujours de l’impatience, ou du mécontentement ET AUSSI de la bonne volonté gâchée (le mec qui passe 1 journée à faire sa PR bien propre et commentée et qui voit 20 ou 30 jours après que le core a changé… et qu’on lui demande de re-coder en fermant sa PR, c’est peu être frustrant, ou le mec qui attend 30 jours et qui voit 50 PR validées et merged et que la sienne ne l’est jamais… c’est peut-être aussi frustrant…)

Toujours est-il que le CONTRIBUTING.md donne déjà quelques infos, peut-être à détailler, ou à « pondérer » selon un nombre de points :
Verbose = OK = 1 point
Structure de la PR = respectée = 2 points
CI = passé = 1 point…

etc… et ça définit une liste de passage devant les yeux des valideurs :happy: Je sais, c’est long, mais y’a bien des génies ici pour automatiser ça ou de bonnes volontés qui aimeraient aider simplement à « verifier » mais qui ne peuvent/veulent pas avoir le pouvoir de valider les PR et les « merged »…
Si on aide les valideurs, les validations iront plus vite :wink:

2 « J'aime »

Merci Hubert, ton premier paragraphe explique bien la frustration ressentie quand on se casse le c… à faire une PR et qu’elle met des semaines à être validée…
Je n’ai pas tout compris à la deuxième partie, sauf qu’il faudrait que je lise un fichier contributing.md :wink: Bon à savoir !