-[ ] J'ai choisi un nom de commit conforme aux [conventions git du projet](conventions-git.md).
-[ ] J'ai choisi un nom de commit conforme aux [conventions git du projet](conventions-git.md).
- [ ] J'ai bien découpé mes changements en plusieurs petits commits atomiques.
- [ ] J'ai bien découpé mes changements en plusieurs petits commits atomiques.
# À vérifier avant de soumettre une "merge request"
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". Pour ce faire, rendez-vous dans [l'onglet "merge request"](https://git.duniter.org/nodes/rust/duniter-rs/merge_requests) du Gitlab.
Avant de soumettre une "merge request", vous devez vérifier que votre branche est bien à jour par rapport à la branche cible (`dev` dans cet example). 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 :
git checkout dev # basculer sur la branche dev
git pull # mettre à jour la branche dev par rapport au dépôt distant
git checkout VOTRE_BRANCHE # basculer à nouveau sur votre branche
git rebase dev # prendre dev comme nouvelle base pour votre branche
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 :
git rebase --abort
S'il n'y a pas de conflits ou si ceux-ci ont été résolus, vous pouvez soumettre 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.
