Intéressant cette histoire de patch. Dans les projets où on est plusieurs, j'estime que tous les patchs doivent avoir été relu par quelqu'un. En général, c'est en post-commit via une mailing liste qui mail un diff de tous les changements.
Ce processus fonctionnait super bien avec svn, mais avec du décentralisé, c'est un poil plus compliqué. De ta description, j'en conclus que tu relis tout ce qui est poussé vers ta branche avant intégration. Et donc tes commits arrivent directement sur cette branche et ne sont pas relus par les autres...
A mon avis, tu devrais te plier au même processus que les autres. Non pas pour t’embêter, mais pour que tes collègues puissent relire tes patch. Personne n'est à l'abri d'une erreur à la con...
[^] # Re:Sécuritédesbackends
Posté par Philippe F (site web personnel) . En réponse à la dépêche Le Weboob nouveau est arrivé. Évalué à 2.
Intéressant cette histoire de patch. Dans les projets où on est plusieurs, j'estime que tous les patchs doivent avoir été relu par quelqu'un. En général, c'est en post-commit via une mailing liste qui mail un diff de tous les changements.
Ce processus fonctionnait super bien avec svn, mais avec du décentralisé, c'est un poil plus compliqué. De ta description, j'en conclus que tu relis tout ce qui est poussé vers ta branche avant intégration. Et donc tes commits arrivent directement sur cette branche et ne sont pas relus par les autres...
A mon avis, tu devrais te plier au même processus que les autres. Non pas pour t’embêter, mais pour que tes collègues puissent relire tes patch. Personne n'est à l'abri d'une erreur à la con...