Coroid génère automatiquement un pull request sur une branche dédiée, par rapport à votre branche de base. Il ne procède pas au merge. Cette décision vous revient entièrement.
Ce qui l'accompagne
- Le changement de code
- Les tests écrits ou mis à jour en parallèle
- Le résultat de l'exécution de votre suite de tests existante
- Le résultat des contrôles de qualité configurés dans votre organisation
- Une analyse du diff avec les anomalies déjà corrigées ou listées
Il ne s'agit pas de remplacer votre revue. Il s'agit simplement que celle-ci ne soit pas la première étape.
Ce qu'il faut réellement vérifier
Les aspects mécaniques ont déjà été vérifiés. Concentrez-vous sur ce que seul vous pouvez juger :
- Est-ce que cela résout le bon problème ? Comparez-le aux critères d'acceptation définis dans la spécification, et non à votre souvenir de ce que vous aviez demandé.
- S'intègre-t-il bien dans la base de code ? Conventions, nommage, structure des abstractions. Coroid analyse votre code pour les déterminer et les applique correctement dans la plupart des cas — mais pas toujours.
- Quels éléments touche-t-il sans que vous l'ayez prévu ? Jetez un œil à la liste des fichiers avant le diff.
- Les nouveaux tests sont-ils pertinents ? Un test qui passe sans valider quoi que ce soit est pire qu'aucun test.
Renvoyer le travail
Vous avez trois options, classées par coût croissant :
- Laisser un commentaire et demander une refonte — la tâche est renvoyée à l'agent développeur avec vos remarques, en conservant son contexte. Idéal pour « c'est correct mais incomplet ».
- Rejeter le plan et relancer le processus — lorsque l'approche choisie est erronée, et non son exécution.
- Annuler la tâche — lorsque le travail n'a tout simplement pas lieu d'être. L'annulation arrête the exécution et libère la place occupée par l'agent.
Soyez précis dans vos commentaires de refonte, comme vous le feriez avec un collègue. « Ceci ne gère pas le cas où parseRange" produit une correction ; « nécessite des améliorations » donne lieu à une
interprétation approximative.
Effectuer le merge
Effectuez le merge via votre outil habituel, en respectant vos règles de protection de branche, les contrôles requis et les validations d'approbation. Coroid ouvre l'pull request ; votre processus existant détermine ce qui en résulte.
Rien concernant le merge n'est particulier — c'est voulu. L'pull request est un élément ordinaire, donc il suit les mêmes procédures de revue et d'intégration continue que celles que vous utilisez déjà.
Suivant
Où aller ensuite — vous guide vers le reste de la documentation selon l'action que vous souhaitez entreprendre.