Ben c'est pas forcément une mauvaise idée…
D'un côté tu laisses plus de libertés aux développeurs, de l'autre tu leurs mets le couteau sous la gorge en liant directement leurs erreurs aux performances client… Difficile de trouver quoi que ce soit de plus impliquant

Après, ce genre de trucs s'inscrit dans une logique de déploiement continu (ce qui correspond généralement à des développements plus courts, souvent plus simples, et donc on peut espérer, moins pénalisants en cas d'erreurs), et ça semble justement être le chemin qu'ils ont suivi, donc personnellement, ça ne me choque pas tant que ça.
(D'ailleurs, la revue de code par quelqu'un d'une autre équipe, c'est un peu du luxe, et je pense pas qu'il y ait *tant* de monde que ça qui en bénéficie dans la population mondiale des développeurs ^^)
(Et personnellement, j'ai tendance à penser que de savoir que ton code va être revu par quelqu'un qui va "forcément voir tes erreurs" ça t'incite plutôt à mal coder qu'à bien coder. Maintenant, c'est aussi bien d'avoir des gardes fous occasionnels, et il faut surtout pas chercher à s'en priver à tout prix ^^)