Re: Validation des PR sur le core

Comment on fait un +1 sur une PR dans Gitea.
Je vois pas les icones moi.

++
Eric


Le mer. 18 mars 2020 à 13:55, Maïeul Rouquette <[email protected]> a écrit :

> 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/
> _______________________________________________
> liste: https://listes.rezo.net/mailman/listinfo/spip-dev
> doc: https://www.spip.net/
> dev: https://core.spip.net/
> irc://irc.freenode.net/spip
>