[docs] développeurs: vérifications avant la demande de fusion d'une merge request
Compare changes
@@ -87,9 +101,31 @@ Pensez bien à préfixer votre commit par `wip:` pour indiquer que c'est un "wor
Après avoir rempli les critères ci-dessus dans vos commits, pour que votre code soit intégré à une branche et mis à disposition des autres contributeurs, il faut soumettre une "merge request". Si ce n'est pas déjà fait, rendez-vous dans [l'onglet "merge request"](https://git.duniter.org/nodes/rust/duniter-rs/merge_requests) du Gitlab. Une merge request ne doit concerner qu'un seul sujet.
Avant de soumettre une "merge request", vous devez vérifier que votre branche est bien à jour par rapport à la branche cible (`dev` dans cet exemple). Comme cette branche avance fréquemment, il est possible que de nouveaux commits aient eu lieu pendant que vous travailliez sur votre branche (nommée VOTRE_BRANCHE, ici). Si c'est le cas ou en cas de doute, pour mettre à jour votre branche par rapport à `dev`, faites comme suit :
En cas de conflits pendant le rebase que vous n'arrivez pas à résoudre, il faut contacter @librelois en lui indiquant le hash du commit sur lequel se base VOTRE_BRANCHE au moment du rebase pour qu'il puisse reproduire le rebase et voir les conflits en question. En attendant sa réponse, vous pouvez annuler le rebase et travailler sur VOTRE_BRANCHE sans se mettre a jour :
S'il n'y a pas de conflits ou si ceux-ci ont été résolus, vous pouvez demander la fusion de votre merge request (dans le cas contraire, ne le faîtes pas car elle ne sera pas acceptée). Une fois la "merge request" émise, une discussion est ouverte spécialement pour celle-ci. Elle vous permettra de discuter des changements que vous avez faits. N'hésitez pas à identifier quelqu'un en écrivant @pseudo pour qu'il soit notifié de votre demande. Ne vous impatientez pas, la relecture de votre contribution peut prendre plus ou moins de temps en fonction de son contenu ! Il est préférable de prendre son temps avant de d'intégrer une nouvelle contribution car une fois intégrée à la branche dev, modifier les commits serait très gênant pour les autres contributeurs. Cela permet de garder un historique des commits clair et compréhensible.
La discussion générale sert à commenter la merge request dans son ensemble, par exemple pour tagger un développeur pour une demande de relecture. Quand il s'agit de discuter un changement précis dans le code, il faut se rendre dans l'onglet "Changes" de la merge request et commenter sous l'extrait de code impliqué. Cela permet de découper plus facilement la résolution des problèmes soulevés par la merge request via la fonctionnalité de "résolution de commentaire".
@@ -103,7 +139,7 @@ Ensuite nettoyez l'historique de votre branche avec un rebase interactif :