Re: Validation des PR sur le core

Le 18/03/2020 à 13:25, nicod_ a écrit :
> Le 17/03/2020 à 22:00, Charles Razack a écrit :
>> 2 à 3 validations à base de +1 me semble un nombre raisonnable aussi au
>> regard du nombre de personnes actives et des disponibilités. J'ai envie
>> d'ajouter quand même : sous réserve qu'il n'y ait pas discussion non
>> résolue en cours sur le PR en question, puisque l'idée reste d'acter un
>> consensus.
>> Et aussi que lâcher un +1 suppose dans la mesure du possible que le PR a
>> été testé, c'est pas juste l'équivalent de dire "ah ouais bonne idée".
> 
> On est d'accords. Partons sur 3 +1 ?
> 
> 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/