Le Théâtre des Pull Requests et de la Revue de Code

L'intervention de Saša Jurić à la Goatmire Elixir Conf a transformé la revue de code en un récit captivant. Il a mis en lumière le problème courant des pull requests (PRs) trop volumineux et complexes, conduisant à des revues superficielles, des risques de sécurité et des bases de code difficiles à maintenir. Conclusion principale : les PRs révisables devraient idéalement prendre 5 à 10 minutes à examiner, idéalement moins de 300 lignes de code. Ceci est obtenu en créant des messages de commit concis et narratifs qui expliquent clairement le raisonnement et les étapes de chaque modification. Décomposer les grandes fonctionnalités en PRs plus petits et utiliser des outils comme `git fixup` pour maintenir un historique de commit propre sont cruciaux pour une revue de code efficace et un code de meilleure qualité. L'intervention a souligné que dire "Je ne comprends pas" est préférable à un "LGTM" sans signification.