• [^] # Re: Review

    Posté par (site web personnel) . En réponse au journal De tout, de rien, des liens, du vrac (des bookmarks quoi). Évalué à 3.

    Les "reviewers" sont-ils des personnes dédiées ou ça fait partie des dev ? En gros les devs font la revue des autres dev ?

    Je viens de regarder un peu plus reviewboard (ça faisait 6 mois que j'en avais un dans un coin…) et en fait c'est plutôt sympa.
    Je ne sais pas trop comment ça marche avec git donc je ne vais parler que de ce que ça donne avec mercurial.
    Déjà, il faut prendre cette extension : https://bitbucket.org/haard/mercurial-reviewboard/overview Il y en a peut-être d'autres mais avec https://bitbucket.org/mdelagra/mercurial-reviewboard/overview j'ai eu des problèmes, et je n'ai pas essayé l'extension initiale http://code.google.com/p/mercurial-reviewboard mais ça vaudrait le coût de voir les différences.

    Bon, quoi qu'il en soit cette extension fonctionne sans problème en ligne de commande (hg postreview tip par exemple) et ça fonctionne aussi dans tortoisehg (avec une interface correcte) donc c'est bien.

    Là où ça m'intéresse beaucoup plus c'est qu'il est possible d'envoyer une review sur toute une branche. Et ça c'est bien.
    Avec un fonctionnement du genre, je peux imaginer un workflow qui serait :

    • création d'une feature branch
    • code / commit / push
    • code / commit / push
    • code / commit / push
    • fonctionnalité / story / … finie
    • demande de review sur la branche
    • review
    • fix / commit (/ push ?)
    • mise à jour de la review avec soit les commits non pushés, soit en resélectionnant les bons
    • OK
    • merge avec le tronc principal
    • close de la feature branch

    Bon c'est approximatif, fait rapidos, mais je pense que ça respecte en gros l'esprit de ce que je souhaiterais.

    Dans un contexte d'entreprise, quand le chef dit, on râle un peu et on fait ;)

    Oué, ça c'est avec des gens civilisés ;-)

    On a un peu rechigné au départ, c'est normal

    Et au final, après un moment, ça donne quoi ? Est-ce que vous reviendriez en arrière ? Est-ce que c'est vu comme un mal obligatoire, comme une perte de temps, ou comme un vrai mieux et une amélioration de la qualité ?
    Et pour finir, le temps "perdu" en code review, est-il vraiment regagné en réduction / suppression de problèmes, bugs, etc ?