[TECH] Merge automatique grâce aux Github Actions

[bpetetot]

🦄 Problème

Le processus de merge des PRs est assez fastidieux actuellement.
Quand on souhaite merger une PR, on doit :

• Prévenir les autres dev que l'on souhaite merger une PR
• Rebase la PR avec la branche dev
• Attendre que le workflow de CI soit ok
• Prier pour que personne n'ai mergé entre temps
• Faire le merge de la PR et changer le titre de merge par défaut avec celui de la PR

🤖 Solution

Automatiser ce processus grâce aux Github Actions.
Nous utilisons un github action qui répond exactement à nos besoins :
https://github.com/pascalgn/automerge-action

Elle permet de déclencher un processus de merge automatique sur le PR ayant été labellisées ":rocket: Ready to merge". Quand la PR a ce label, l'action va:

• Checkout la branche
• Vérifier que les checks obligatoires sont passés et ok
• Vérifier que la PR respecte les conditions d'approbation
• Rebaser la PR avec la branche cible (dev)
  • Si KO => un check d'erreur apparait dans la PR (car conflit au rebase)
  • Si OK => merge la branche dans la branche cible (dev) et change le titre du commit de merge avec le nom de la PR, et enfin ferme la PR

🌈 Remarques

Conditions à respecter pour que la PR soit merge :

• the required number of review approvals has been given (if enabled in the branch protection rules) and
• the required checks have passed (if enabled in the branch protection rules) and
• the pull request is up to date (if enabled in the branch protection rules)

💯 Pour tester

Vous pouvez tester sur ce repo de test

1. Créer une PR
2. Mettre le label ":rocket: Ready to merge"
3. Faire approuver la PR

[pix-service]

I'm deploying this PR to these urls:

• App: https://app-pr1456.review.pix.fr
• Orga: https://orga-pr1456.review.pix.fr
• Certif: https://certif-pr1456.review.pix.fr
• Admin: https://admin-pr1456.review.pix.fr
• API: https://api-pr1456.review.pix.fr/api/

Please check it out!

[jbuget]

Qu'est-ce qui se passe si la branche de la PR n'a pas démarré sur dev mais sur une autre branche (ex : la branche de la PR #1434 était initialement basée sur la branche de la PR #1433) ?

[jbuget]

Comment fait-on pour relancer un auto-merge après qu'un premier auto-merge a échoué (pour cause de conflits de merge par exemple) ?

[MelanieMEB]

L'auto-rebase en cas de décalage entre dev et la branche est pratique, mais c'est un système que nous n'avons jamais mis en place actuellement pour éviter des rebase auto qui ne correspondent pas à ce que l'on souhaite.
Je serais pour laisser l'auto-merge quand la branche est à jour avec dev, mais continuer de laisser les dev rebase eux mêmes.

[jonathanperret]

L'auto-rebase en cas de décalage entre dev et la branche est pratique, mais c'est un système que nous n'avons jamais mis en place actuellement pour éviter des rebase auto qui ne correspondent pas à ce que l'on souhaite.

Pour que je comprenne, est-ce que tu aurais stp un exemple de rebase automatique qui ne correspondrait pas à ce que l'on souhaite ?

[bpetetot]

@jbuget

Qu'est-ce qui se passe si la branche de la PR n'a pas démarré sur dev mais sur une autre branche (ex : la branche de la PR #1434 était initialement basée sur la branche de la PR #1433) ?

Si on applique le label "Ready to merge" alors la branche sur cette PR alors la branche #1434 sur rebase sur la "Base branch" qui est #1433 et ensuite elle sera mergée sur cette dernière.

Comment fait-on pour relancer un auto-merge après qu'un premier auto-merge a échoué (pour cause de conflits de merge par exemple) ?

Il faut rebase en local, résoudre les conflits et push à nouveau sur la branche. (comme dans le process actuel) et supprimer puis remettre le label.

@MelanieMEB @jonathanperret

L'auto-rebase en cas de décalage entre dev et la branche est pratique, mais c'est un système que nous n'avons jamais mis en place actuellement pour éviter des rebase auto qui ne correspondent pas à ce que l'on souhaite.
Je serais pour laisser l'auto-merge quand la branche est à jour avec dev, mais continuer de laisser les dev rebase eux mêmes.

Je ne pense pas qu'on puisse avoir des soucis particulier avec l'auto-rebase car :

• il sera déclenché uniquement quand le branche aura le label "Ready to merge"
• il échouera s'il y a un conflit au rebase (il faudra alors le résoudre manuellement comme avant)

Je suis en train d'approfondir certains cas particulier pour voir si ça pourrait poser problème dans notre process actuel.