Ah, et une petite chose encore : Si un ticket est affecté à quelqu'un, avant de faire une correction au sujet du ticket, voyez d'abord avec cette personne pour qu'il n'y ait pas conflit ou incompréhension, voire plusieurs soumissions redondantes de code.
Le 3 août 2013 20:29, Franck Paul <[email protected]> a écrit : > Bonsoir, > > Quelques remarques au sujet des PR (Pull-Request) faits sur les dépôts > Dotclear et Clearbricks sur Bitbucket : > > Il serait bien de : > > - commencer à résoudre un ticket que si la discussion à son sujet est > terminée **et** qu'un consensus a été trouvé. > > - faire un PR uniquement d'ensemble cohérent et pas de commits pris les > uns après les autres, ça rend le contrôle du code beaucoup plus complexe et > ça disperse les éventuels commentaires que nous (les relecteurs) pourraient > avoir à faire. > > - veiller à faire les tests idoines une fois le code complété. > > À noter : > > - les contributeurs peuvent signaler leurs commits faits sur leurs dépôts > via la ML-dev et demander qu'un ou plusieurs relecteurs vienne jeter un > œil, répondre à une question en suspens, fournir une ou des traductions > manquantes, … Toujours dans l'optique d'obtenir finalement un PR cohérent. > > C'est de mon point de vue la seule façon de conserver un code clair, > propre et cohérent au sein du core de Dotclear et Clearbricks. > > M > erci d'avance de faciliter le travail des relecteurs ! > > -- > Franck > -- Franck
-- Dev mailing list - [email protected] - http://ml.dotclear.org/listinfo/dev
