417
edits
LTS 18 ACTIVITY 2025: Difference between revisions
Jump to navigation
Jump to search
no edit summary
Eric.seigne (talk | contribs) No edit summary |
Eric.seigne (talk | contribs) No edit summary |
||
| Line 36: | Line 36: | ||
*pour nous il suffit de suivre le message pour bien comprendre le bug | *pour nous il suffit de suivre le message pour bien comprendre le bug | ||
Et en plus ça enrichi la "documentation" implicite du projet<span> </span>: si 2 ans plus tard vous revenez sur une PR vous aurez tous les détails / éléments de contexte pour comprendre pourquoi une correction a été faite et quelle option avait été prise. | Et en plus ça enrichi la "documentation" implicite du projet<span> </span>: si 2 ans plus tard vous revenez sur une PR vous aurez tous les détails / éléments de contexte pour comprendre pourquoi une correction a été faite et quelle option avait été prise.Exemple de PR complexes et du domaine lié: | ||
Exemple de PR complexes et du domaine lié: | |||
*https://github.com/Dolibarr/dolibarr/pull/33859 -> compta | *https://github.com/Dolibarr/dolibarr/pull/33859 -> compta | ||
*https://github.com/Dolibarr/dolibarr/pull/36553 -> marges et arrondis | *https://github.com/Dolibarr/dolibarr/pull/36553 -> marges et arrondis | ||
==== Bonne pratique pour vos prochaines PR ==== | ====Bonne pratique pour vos prochaines PR==== | ||
Exemple d'une PR "idéale" pour nous: elle est documentée et nous permet de comprendre rapidement le contexte et de reproduire la situation https://github.com/Dolibarr/dolibarr/pull/34511 | Exemple d'une PR "idéale" pour nous: elle est documentée et nous permet de comprendre rapidement le contexte et de reproduire la situation https://github.com/Dolibarr/dolibarr/pull/34511 | ||
D'autre part quelques automatismes ont été mis en place pour aider le travail des co-mergeurs<span> </span>: | D'autre part quelques automatismes ont été mis en place pour aider le travail des co-mergeurs<span> </span>: | ||