Pas trop chiant comme contrainte dans les processus de dev ?
On a des scripts maisons pour pusher le diff sur le board avant le commit et on peut décider d'une politique "permissive" (reviewer et commiter quand même et attendre les remarques des copains pour s'améliorer) ou bien bloquante (ne commiter que lorsqu'on a eu les autorisations de N collègues).
Le bloquage peut être automatique ou bien on peut aussi s'auto-discipliner, ce qui fonctionne assez bien dans une équipe de taille modeste. De toute façon si tu commits sans attendre les autorisations, ça se voit et tu te fais vite taper sur les doigts.
Je l'utilise de manière "permissive" au sein d'une équipe et les review se font en opportuniste (quand les gens ont le temps) et même dans ce cas là, ça permet de diffuser un peu la connaissance, les gens sont en général assez contents de pouvoir donner leur avis sur le code des autres.
Il est important que l'outil de review soit assez complet pour pouvoir commenter facilement n'importe quelle portion de code soumise, d'offrir un système de "ping-pong" de commentaires, Review Board le fait et permet également de pouvoir engager un thread de discussion de façon hiérarchisée. Mais je pense que les autres outils doivent également le proposer.
Perso j'adore et c'est venu s'insérer assez naturellement dans notre processus de développement.
# Review
Posté par Guillaume Denry (site web personnel) . En réponse au journal De tout, de rien, des liens, du vrac (des bookmarks quoi). Évalué à 4.
Oui.
Oui.
Review Board.
On a des scripts maisons pour pusher le diff sur le board avant le commit et on peut décider d'une politique "permissive" (reviewer et commiter quand même et attendre les remarques des copains pour s'améliorer) ou bien bloquante (ne commiter que lorsqu'on a eu les autorisations de N collègues).
Le bloquage peut être automatique ou bien on peut aussi s'auto-discipliner, ce qui fonctionne assez bien dans une équipe de taille modeste. De toute façon si tu commits sans attendre les autorisations, ça se voit et tu te fais vite taper sur les doigts.
Je l'utilise de manière "permissive" au sein d'une équipe et les review se font en opportuniste (quand les gens ont le temps) et même dans ce cas là, ça permet de diffuser un peu la connaissance, les gens sont en général assez contents de pouvoir donner leur avis sur le code des autres.
Il est important que l'outil de review soit assez complet pour pouvoir commenter facilement n'importe quelle portion de code soumise, d'offrir un système de "ping-pong" de commentaires, Review Board le fait et permet également de pouvoir engager un thread de discussion de façon hiérarchisée. Mais je pense que les autres outils doivent également le proposer.
Perso j'adore et c'est venu s'insérer assez naturellement dans notre processus de développement.