librelist archives

« back to archive

Gestion des Pull Request

Gestion des Pull Request

From:
Alexis Metaireau
Date:
2014-01-13 @ 08:28
Salut,

C'est trop bien de voir qu'on se motive à bosser sur Daybed; On va
peut-être réussir à avoir quelque chose de potable :-)

Par contre, ça fait plusieurs fois qu'on se rends compte qu'on aurait
besoin de plus de discussion sur les Pull Request alors qu'elles ont été
intégrées à la branche "dev". Peut-être qu'il nous faut un process un
peu plus cadré pour la gestion de celles ci?

Je sais que sur le projet scikits.learn, par exemple, ils choisissent
d'avoir au moins 2 personnes qui review un code avant de l'intégrer,
qu'est-ce que vous en dites ?

Si vous avez d'autres propositions, elles sont les bienvenues,
évidemment !

Alexis

Re: [daybed.dev] Gestion des Pull Request

From:
Rémy Hubscher
Date:
2014-01-13 @ 08:33
On peut dire qu'elle doivent être mergée par quelqu'un d'autres que celui
qui les ouvre ?
Le 13 janv. 2014 09:28, "Alexis Metaireau" <alexis@notmyidea.org> a écrit :

> Salut,
>
> C'est trop bien de voir qu'on se motive à bosser sur Daybed; On va
> peut-être réussir à avoir quelque chose de potable :-)
>
> Par contre, ça fait plusieurs fois qu'on se rends compte qu'on aurait
> besoin de plus de discussion sur les Pull Request alors qu'elles ont été
> intégrées à la branche "dev". Peut-être qu'il nous faut un process un
> peu plus cadré pour la gestion de celles ci?
>
> Je sais que sur le projet scikits.learn, par exemple, ils choisissent
> d'avoir au moins 2 personnes qui review un code avant de l'intégrer,
> qu'est-ce que vous en dites ?
>
> Si vous avez d'autres propositions, elles sont les bienvenues,
> évidemment !
>
> Alexis
>

Re: [daybed.dev] Gestion des Pull Request

From:
Alexis Metaireau
Date:
2014-01-13 @ 08:57
Dans le cas de la seconde pull request à laquelle je pense ça ne serait
pas suffisant: [1]https://github.com/spiral-project/daybed/pull/125
(Mathieu l'à ouverte et tu l'a mergée)





Le Lun 13 Janvier 2014, à 9:33, Rémy Hubscher a écrit :

  On peut dire qu'elle doivent être mergée par quelqu'un d'autres que
  celui qui les ouvre ?

Le 13 janv. 2014 09:28, "Alexis Metaireau" <[2]alexis@notmyidea.org> a
écrit :

Salut,



C'est trop bien de voir qu'on se motive à bosser sur Daybed; On va

peut-être réussir à avoir quelque chose de potable :-)



Par contre, ça fait plusieurs fois qu'on se rends compte qu'on aurait

besoin de plus de discussion sur les Pull Request alors qu'elles ont
été

intégrées à la branche "dev". Peut-être qu'il nous faut un process un

peu plus cadré pour la gestion de celles ci?



Je sais que sur le projet scikits.learn, par exemple, ils choisissent

d'avoir au moins 2 personnes qui review un code avant de l'intégrer,

qu'est-ce que vous en dites ?



Si vous avez d'autres propositions, elles sont les bienvenues,

évidemment !



Alexis

References

1. https://github.com/spiral-project/daybed/pull/125
2. mailto:alexis@notmyidea.org

Re: [daybed.dev] Gestion des Pull Request

From:
Rémy Hubscher
Date:
2014-01-13 @ 14:56
Oui après c'est un cas extrême qui ne devrait pas se reproduire si souvent.
Dans ce cas je pensais vraiment juste fixer les tests avant de merger. 
My bad.
Après est-ce un problème de revert tel qu'on l'a fait ?

Pour moi la branche dev n'est pas tout à fait encore la branche master 
mais je ferais plus attention à l'avenir.

Rémy

Le 13/01/2014 09:57, Alexis Metaireau a écrit :
> Dans le cas de la seconde pull request à laquelle je pense ça ne 
> serait pas suffisant: 
> https://github.com/spiral-project/daybed/pull/125 (Mathieu l'à ouverte 
> et tu l'a mergée)
> Le Lun 13 Janvier 2014, à 9:33, Rémy Hubscher a écrit :
>>
>> On peut dire qu'elle doivent être mergée par quelqu'un d'autres que 
>> celui qui les ouvre ?
>>
>> Le 13 janv. 2014 09:28, "Alexis Metaireau" <alexis@notmyidea.org 
>> <mailto:alexis@notmyidea.org>> a écrit :
>>
>>     Salut,
>>     C'est trop bien de voir qu'on se motive à bosser sur Daybed; On va
>>     peut-être réussir à avoir quelque chose de potable :-)
>>     Par contre, ça fait plusieurs fois qu'on se rends compte qu'on aurait
>>     besoin de plus de discussion sur les Pull Request alors qu'elles
>>     ont été
>>     intégrées à la branche "dev". Peut-être qu'il nous faut un process un
>>     peu plus cadré pour la gestion de celles ci?
>>     Je sais que sur le projet scikits.learn, par exemple, ils choisissent
>>     d'avoir au moins 2 personnes qui review un code avant de l'intégrer,
>>     qu'est-ce que vous en dites ?
>>     Si vous avez d'autres propositions, elles sont les bienvenues,
>>     évidemment !
>>     Alexis
>>

Re: [daybed.dev] Gestion des Pull Request

From:
Alexis Metaireau
Date:
2014-01-16 @ 09:54





Le Lun 13 Janvier 2014, à 15:56, Rémy HUBSCHER a écrit :

Oui après c'est un cas extrême qui ne devrait pas se reproduire si
souvent.

Dans ce cas je pensais vraiment juste fixer les tests avant de merger.
My bad.

Après est-ce un problème de revert tel qu'on l'a fait ?



Ce n'est pas de cette PR là dont je parles en fait :) see
[1]https://github.com/spiral-project/daybed/pull/125

Peut-être qu'on peut effectivement rester sur le système qu'on à en
place et aviser plus tard si besoin à nouveau, j'ai pas trop envie
d'ajouter des procédures si elles sont pas necessaires.



Alexis

References

1. https://github.com/spiral-project/daybed/pull/125