>Surtout que si tu mets des hooks bien coercitifs sur le gestionnaire de version (genre ça commit pas si la couverture par les tests n'est pas totale, si le codestyle n'est pas parfaitement respecté etc)
Impossible à mettre de tel hook sur un commit. Vu qu'il faut 40mn pour compiler un gecko, et je ne sais pas combien pour lancer tout les tests, je vois mal attendre le dev pendant une heure que son commit se finisse.
La vérification du coding style et les tests sont faits en amonts, avant le commit. Un développeur n'a le droit de commiter son patch que si il a passé avec succés au moins deux reviews (deux vérifications faites par deux responsables de la partie du code en question, vérifications faites au niveau du coding style, du fonctionnement, de la qualité du code pour éviter les leaks etc ). Et après le commit, il y a les tinderbox (ex, celle pour firefox 3 : http://tinderbox.mozilla.org/showbuilds.cgi?tree=Firefox ) qui compilent en permanence Gecko sur plein de bécanes différentes (sous linux, windows mac etc), et réalisent les tests fonctionnelles et les tests de perfs. Et si la compile passe pas (parce que les devs n'ont pas toutes les machines sur leur bureau pour tester la compile sur toutes les archis avant le commit), ou si il y a des tests qui passent pas, le commiter est prié de retirer son patch de suite et de corriger le problème.
[^] # Re: FUD
Posté par Laurent J (site web personnel, Mastodon) . En réponse au journal La fin de Thunderbird ?. Évalué à 2.
Impossible à mettre de tel hook sur un commit. Vu qu'il faut 40mn pour compiler un gecko, et je ne sais pas combien pour lancer tout les tests, je vois mal attendre le dev pendant une heure que son commit se finisse.
La vérification du coding style et les tests sont faits en amonts, avant le commit. Un développeur n'a le droit de commiter son patch que si il a passé avec succés au moins deux reviews (deux vérifications faites par deux responsables de la partie du code en question, vérifications faites au niveau du coding style, du fonctionnement, de la qualité du code pour éviter les leaks etc ). Et après le commit, il y a les tinderbox (ex, celle pour firefox 3 : http://tinderbox.mozilla.org/showbuilds.cgi?tree=Firefox ) qui compilent en permanence Gecko sur plein de bécanes différentes (sous linux, windows mac etc), et réalisent les tests fonctionnelles et les tests de perfs. Et si la compile passe pas (parce que les devs n'ont pas toutes les machines sur leur bureau pour tester la compile sur toutes les archis avant le commit), ou si il y a des tests qui passent pas, le commiter est prié de retirer son patch de suite et de corriger le problème.