Re: Validation des PR sur le core

Hop,

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 ?
> 

Deux suffisent amha, je me répète mais d'expérience j'ai un paquet de 
patchs qui ont attendu ne serait-ce qu'un seul retour sur redmine, donc 
je pense qu'on est pas assez nombreux à être réactifs pour espérer 3 
retours.

> 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 préfère qu'on évite d'avoir des commits de merge dans la 
branche master, pour le reste, peu importe, ça dépend un peu du contenu 
de la PR. Si celle-ci est un travail de longue durée il faut garder tous 
ses commits, si c'est une simple patch avec plus de oups que de commits 
utiles un squash ne fait pas de mal, etc.

++
b_b