Avec Stash et la nouvelle fonctionnalité de Pull Requests, les revues de code sont désormais partie intégrante de votre process de développement. Les développements sont effectués sur des branches et lorsque le code est prêt à être mergé dans la branche principale une Pull Request est ouverte. Le code n’intègre la branche que quand il a fait l’objet d’une revue dans une Pull Request. Tous les bénéfices de la revue de code sont ainsi inclus dans votre workflow !
Créer une Pull Request ressemble à créer une discussion. Les participants de la revue peuvent voir les changements effectués, les commenter et commiter d’autres changements et amélioration sur la branche si nécessaire. Une fois que tout le monde a validé, la branche peut alors être mergée au trunk ou à la branche principale.
Il n’a jamais été aussi simple de lancer une revue sur son code, il suffit de cliquer sur le bouton “Pull Request” dans le header du repository, sélectionner la branche sur laquelle on a travaillé, la branche que l’on souhaite intégrer et d’ajouter une courte description de ce qui a été fait.
Les discussions
L’apport principal d’une pull request est la discussion qui est lancée autour des changements de code que l’on fait. La vue d’ensemble présente de façon exhaustive l’activité autour de la pull request : les commentaires sur les diff, les réponses, et les nouveaux commits sur la branche.
Les fichiers modifiés
Les diffs d’une pull request Stash mettent en évidence les changements qui résulteront du merge. L’arborescence des diffs montre les fichiers modifiés, il suffit de cliquer sur l’un d’entre eux pour voir précisément ce qui a été modifié, ajouté ou supprimé. Les flux de commentaires dans le diff permettent de contextualiser les discussions autour de votre code.
Les commits
Les pull requests sont dynamiques. Elles permettent non seulement de créer une discussion en temps réel autour des changements du code, mais encore de commiter de nouveau changements sur la branche. Stash met à jour automatiquement l’onglet “Commits” de la pull request pour vous permettre de voir exactement quels commits seront mergés. Les commentaires sont automatiquement repositionnés lorsque des lignes sont ajoutées ou supprimées. Si un ligne faisant l’objet d’un commentaire venait à être supprimée, le commentaire reste accessible dans l’activité avec la mention “outdated” pour vous informer que ce fragment de code a été modifié dans des commits récents.
Les notifications
Si quelqu’un vous a ajouté à la liste des participants de la revue, a commenté la pull request ou mergé vos changements, Stash vous garde informé avec des notifications par mail pour les pull requests vous concernant.
