openerp-community team mailing list archive

[Nhomar Hernández <nhomar@xxxxxxxxx>]

Pedro.

It is a "good to have" but I think we can just pull locally test and then
push.

If you do this process recommended it ia good too! But IMHO Not the rule
for now.

Be careful! The pr button is dangerous without a correct CIS configured.

Written from my android
On Jul 2, 2014 3:16 PM, "Pedro Manuel Baeza Romero" <pedro.baeza@xxxxxxxxx>
wrote:

> We have to set new contributing HowTos for working with GitHub, and this
> article points out some troubles with Merge PR button, but the way it says
> to merge PR can be very laborious for maintainers. Can we assume this
> amount of extra work? One of the highlights we talk about when the switch
> to GitHub was the Merge PR button.
>
> Regards.
>
>
> 2014-06-27 14:23 GMT+02:00 Tymoteusz Motylewski <t.motylewski@xxxxxxxxxxxx
> >:
>
>> +1 for the clean git history. Merge commits are painful.
>> With clean commit messages and linear history things like changelog can
>> be easily generated automatically. And by changelog I mean sth which really
>> gives the reader some knowledge.
>> See other OSS project TYPO3 Neos for example of such a history
>>
>> http://docs.typo3.org/neos/TYPO3NeosDocumentation/Appendixes/ChangeLogs/102.html
>>
>>
>>
>>
>> 2014-06-27 13:51 GMT+02:00 Lionel Sausin <ls@xxxxxxxxxxxxxxxx>:
>>
>> Very interesting reading. I totally support this point of view even if
>>> I'm only a contributor - clean history is hours not lost searching for
>>> stuff. Bisect, annotate, etc all become easier.
>>>
>>> Le 27/06/2014 12:10, Alexandre Fayolle a écrit :
>>>
>>>  Hello,
>>>>
>>>> I found this blog post both interesting and useful :
>>>> http://blog.spreedly.com/2014/06/24/merge-pull-request-
>>>> considered-harmful
>>>>
>>>> Leonardo Pistone had shown me https://github.com/github/hub before and
>>>> it's a nice tool. And since odoo and the community addons are moving to
>>>> a new platform, I thought I'd share this piece of insight with all of
>>>> you.
>>>>
>>>>
>>>
>>> _______________________________________________
>>> Mailing list: https://launchpad.net/~openerp-community
>>> Post to     : openerp-community@xxxxxxxxxxxxxxxxxxx
>>> Unsubscribe : https://launchpad.net/~openerp-community
>>> More help   : https://help.launchpad.net/ListHelp
>>>
>>
>>
>>
>> --
>>
>> Pozdrawiam serdecznie / Best regards
>> Tymoteusz Motylewski
>> Head of development
>> +48 502 472 202
>>
>> t.motylewski@xxxxxxxxxxxx
>>
>> http://www.macopedia.pl/
>> Ul. Roosevelta 5/8
>> 60-829 Poznań
>> Tel. +48 61 622 96 95
>> Fax +48 61 623 24 48/
>> Macopedia Spółka z ograniczoną odpowiedzialnością, z siedzibą ul.
>> Roosevelta 5/8; 60-829 Poznań, wpisana do Krajowego Rejestru Sądowego,
>> prowadzonego przez Sąd Rejonowy, VIII Wydział Gospodarczy, KRS
>> 0000416744, o opłaconym kapitale zakładowym w wysokości 5.000,00 zł, o
>> numerze NIP: 781-187-78-41, numerze Regon 302076699
>>
>> _______________________________________________
>> Mailing list: https://launchpad.net/~openerp-community
>> Post to     : openerp-community@xxxxxxxxxxxxxxxxxxx
>> Unsubscribe : https://launchpad.net/~openerp-community
>> More help   : https://help.launchpad.net/ListHelp
>>
>>
>
> _______________________________________________
> Mailing list: https://launchpad.net/~openerp-community
> Post to     : openerp-community@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~openerp-community
> More help   : https://help.launchpad.net/ListHelp
>
>