Re: Validation des PR sur le core

Le 18/03/2020 à 13:54, Maïeul Rouquette a écrit :
>> Et pour le mode de fusion, quelle stratégie ?
>>
>> - Fusionner la demande d'ajout
>> - Rebase et fusionner
>> - Rebase et Fusion (--no-ff)
>> - Squash et fusionner
>>
>> Je suis plutôt pour fusionner (merge) systématiquement, pour garder 
>> une trace du travail fait sur une branche parallèle.
>> Votre avis ?
>>
> 
> perso je dirais : il faut un commit de fusion. Après rien n'empêche de 
> "nettoyer" la branche avant la fusion (pour reéécrire les oups), même si 
> cela ne me semble pas indispensable.
> 
> En tout cas, il me semble plus pertinent d'avoir un commit de fusion 
> qu'uen reprise directe de la branche, surtout si celle-ci contient des 
> essais / erreurs
> 
> Un article pour éclairer mon avis
> 
> https://delicious-insights.com/fr/articles/bien-utiliser-git-merge-et-rebase/ 

En fait le mieux serait peut être que l'auteur·e de la PR fasse soi même 
la fusion, en connaissance du contexte et du travail réalisé (simple 
correction, ticket ou travail spécifique), une fois que deux +1 ont été 
ajoutés et qu'il n'y a plus de discussion en cours.

-- 
nicod_