Groupe de revue de code

20 views
Skip to first unread message

Guirec Corbel

unread,
Apr 1, 2014, 7:47:20 AM4/1/14
to rails...@googlegroups.com
Salut les amis,

Je viens d'avoir une idée. Pensez-vous que ça serait possible de faire un groupe de 2 à 5 personnes qui regarderait les commits de chacun et commenteraient le travail réalisé. Il faudrait faire des revues de code régulièrement. 

Qu'en pensez-vous?

Cyril Mougel

unread,
Apr 1, 2014, 7:50:56 AM4/1/14
to ML Rails France
Fait de l'open source. C'est ce qu'il se passe en permanence ...
> --
> --
> Vous avez reçu ce message, car vous êtes abonné au groupe "Railsfrance" de
> Google Groups.
> Pour transmettre des messages à ce groupe, envoyez un e-mail à l'adresse
> rails...@googlegroups.com
> Pour résilier votre abonnement envoyez un e-mail à l'adresse
> railsfrance...@googlegroups.com
> ---
> Vous recevez ce message, car vous êtes abonné au groupe Google Groupes
> "Railsfrance".
> Pour vous désabonner de ce groupe et ne plus recevoir d'e-mails le
> concernant, envoyez un e-mail à l'adresse
> railsfrance...@googlegroups.com.
> Pour obtenir davantage d'options, consultez la page
> https://groups.google.com/d/optout.



--
Cyril Mougel
http://blog.shingara.fr

Guirec Corbel

unread,
Apr 1, 2014, 8:41:34 AM4/1/14
to rails...@googlegroups.com
Euh.... C'est pas faux. Je parlais surtout d'un groupe qui regarderait des projets plus spécifique. J'en ai qui ne sont pas forcement utilisable pour le monde entier. Ceci dit, une solution serait de faire des articles sur mon blog expliquant ce que j'ai fait et pourquoi.


Pour plus d'options, visitez le site https://groups.google.com/d/optout .

Yann Armand

unread,
Apr 1, 2014, 12:54:53 PM4/1/14
to rails...@googlegroups.com
L'idée de la publication sur ton blog est bien pour 2 raisons:
1- en l'écrivant tu va probablement remettre en cause tes assomptions et entamer du refactoring.
2- tu recevra des retour
Mais a mon avis cela ne marche que sur des points spécifiques.

Dans notre equipe nous faisons depuis quelques temps des pairing sessions en fin de projet avec un dev qui n'y a pas participer.
On bénéficie d'un regard neuf qui te met en cause les certitude accumulées pendant le projet. Il faut expliquer et 'refactorer' tout ce qui n'est pas evidant. A la fin le code est souvent grandement améliorer en maintenabilité.

Tout cela pour dire que je suis d'accord avec ton idee de groupe de revue, mais plutot en mode pairing.

Guirec Corbel

unread,
Apr 1, 2014, 1:06:36 PM4/1/14
to rails...@googlegroups.com
Le pairing me va aussi. Peut-être qu'on pourrait organiser une heure par semaine?


Yann Armand

unread,
Apr 1, 2014, 1:14:48 PM4/1/14
to rails...@googlegroups.com

1h par semaine c'est un bon ratio.
Je fait ça avec une collègue américaine et c'est pas mal sur des points spécifique. Ou pour une revue de code globale puis chacun repars travailler de son cote sur ce code.
Avec l'habitude, on sais quelle quantité de code on peut raisonnablement regarder et refactorer de façon efficace dans laps de temps.

Ceci étant dit, ce que je conseiller ici, c'est juste pour faire mon malin, je n'ai actuellement aucun créneaux de libre pour participer a ce genre d'expérience. Désolé.

Vous recevez ce message, car vous êtes abonné à un sujet dans le groupe Google Groupes "Railsfrance".
Pour vous désabonner de ce sujet, visitez le site https://groups.google.com/d/topic/railsfrance/rTT_2RGBhpo/unsubscribe.
Pour vous désabonner de ce groupe et de tous ses sujets, envoyez un e-mail à l'adresse railsfrance...@googlegroups.com.

Guirec Corbel

unread,
Apr 1, 2014, 1:18:02 PM4/1/14
to rails...@googlegroups.com
Anyone else?


Vous recevez ce message, car vous êtes abonné au groupe Google Groupes "Railsfrance".

Pour vous désabonner de ce groupe et ne plus recevoir d'e-mails le concernant, envoyez un e-mail à l'adresse railsfrance...@googlegroups.com.

Simon Courtois

unread,
Apr 1, 2014, 5:53:07 PM4/1/14
to rails...@googlegroups.com
Salut Guirec,

J'ai un peu de temps en ce moment, si tu veux on peut faire une heure d'essai la semaine prochaine et voir ce que ça donne ;)

Simon Courtois

Guirec Corbel

unread,
Apr 4, 2014, 9:32:12 AM4/4/14
to rails...@googlegroups.com
Es-tu disponible ce weekend? Donne moi tes heures et on fera un Hangout si ça te va.

Simon Courtois

unread,
Apr 4, 2014, 9:33:17 AM4/4/14
to rails...@googlegroups.com, Guirec Corbel

Est-ce que tu as plutôt de la dispo en semaine ?

Je t’avoue que je suis plus libre en semaine que le we :/

Simon Courtois
Reply all
Reply to author
Forward
0 new messages