[Freeswitch-users] How to test a Pull Request! aka how to help us out!

Brian West brian at freeswitch.org
Fri Oct 2 17:22:53 MSD 2015


Lol, yea what could possibly go wrong! All is a real bad idea!

On Friday, October 2, 2015, Stanislav Sinyagin <ssinyagin at gmail.com> wrote:

> no, it's definitely a bad idea to merge all pull requests automatically.
> Many of them are in reviewing and criticizing process, and some of them
> weren't yet reviewed.
> Some are known to break things, and aren't yet declined.
>
>
>
> On Fri, Oct 2, 2015 at 9:49 AM, Sergey Safarov <s.safarov at gmail.com
> <javascript:_e(%7B%7D,'cvml','s.safarov at gmail.com');>> wrote:
>
>> Hello Brian
>> I learned to use a little git, and now I can help test requests.
>> I found that before the command "git pr <PR_NUMBER>" must be given "git
>> fetch origin".
>>
>> I would also like suggest to create a branch named "merged_pr" and a
>> script that will merge all open pull requests. So the community will have
>> the opportunity to test the assembly with all current pull requests. If
>> bugs are found, then find them via the "git bisect".
>>
>> If you think it is useful, whether to suggest the implementation of the
>> script?
>>
>> Sergey
>>
>> On Thu, Jul 9, 2015 at 12:04 AM, Brian West <brian at freeswitch.org
>> <javascript:_e(%7B%7D,'cvml','brian at freeswitch.org');>> wrote:
>>
>>> Checkout freeswitch master first!
>>>
>>> git clone https://freeswitch.org/stash/scm/fs/freeswitch.git
>>> freeswitch.git
>>>
>>> cd freeswitch.git
>>>
>>> git config --add remote.origin.fetch
>>> '+refs/pull-requests/*/from:refs/remotes/origin/pr/*'
>>> git config alias.pr '!f() { git checkout pr/$1; git config
>>> branch.pr/$1.merge refs/pull-requests/$1/from;} ; f'
>>>
>>> git pr <PR_NUMBER>
>>>
>>> build, test, report back.
>>>
>>> You can view the list of pull requests here:
>>>
>>> https://freeswitch.org/stash/projects/FS/repos/freeswitch/pull-requests
>>>
>>> Questions? Comments?
>>>
>>> Thanks,
>>>
>>>
>>> --
>>>
>>> *Brian West*
>>> brian at freeswitch.org
>>> <javascript:_e(%7B%7D,'cvml','brian at freeswitch.org');>
>>>
>>>
>>> *Twitter: @FreeSWITCH , @briankwest*
>>> http://www.freeswitchbook.com
>>> http://www.freeswitchcookbook.com
>>>
>>> ClueCon 2015 Call for Speakers
>>> <https://www.cluecon.com/call-for-speakers/> | Register
>>> <https://freeswitch.com/cart.php?gid=1> TODAY! | Reddit: /r/freeswitch
>>> <https://www.reddit.com/r/freeswitch>
>>>
>>> *T:*+19184209001 | *F:*+19184209002 | *M:*+1918424WEST (9378)
>>> *iNUM:*+883 5100 1420 9001 | *ISN:*410*543 | *Skype:*briankwest
>>>
>>> _________________________________________________________________________
>>> Professional FreeSWITCH Consulting Services:
>>> consulting at freeswitch.org
>>> <javascript:_e(%7B%7D,'cvml','consulting at freeswitch.org');>
>>> http://www.freeswitchsolutions.com
>>>
>>> Official FreeSWITCH Sites
>>> http://www.freeswitch.org
>>> http://confluence.freeswitch.org
>>> http://www.cluecon.com
>>>
>>> FreeSWITCH-users mailing list
>>> FreeSWITCH-users at lists.freeswitch.org
>>> <javascript:_e(%7B%7D,'cvml','FreeSWITCH-users at lists.freeswitch.org');>
>>> http://lists.freeswitch.org/mailman/listinfo/freeswitch-users
>>> UNSUBSCRIBE:http://lists.freeswitch.org/mailman/options/freeswitch-users
>>> http://www.freeswitch.org
>>>
>>
>>
>> _________________________________________________________________________
>> Professional FreeSWITCH Consulting Services:
>> consulting at freeswitch.org
>> <javascript:_e(%7B%7D,'cvml','consulting at freeswitch.org');>
>> http://www.freeswitchsolutions.com
>>
>> Official FreeSWITCH Sites
>> http://www.freeswitch.org
>> http://confluence.freeswitch.org
>> http://www.cluecon.com
>>
>> FreeSWITCH-users mailing list
>> FreeSWITCH-users at lists.freeswitch.org
>> <javascript:_e(%7B%7D,'cvml','FreeSWITCH-users at lists.freeswitch.org');>
>> http://lists.freeswitch.org/mailman/listinfo/freeswitch-users
>> UNSUBSCRIBE:http://lists.freeswitch.org/mailman/options/freeswitch-users
>> http://www.freeswitch.org
>>
>
>

-- 

*Brian West*
brian at freeswitch.org


*Twitter: @FreeSWITCH , @briankwest*
http://www.freeswitchbook.com
http://www.freeswitchcookbook.com

Got Bugs? Report them here <https://freeswitch.org/jira>! | Reddit:
/r/freeswitch <https://www.reddit.com/r/freeswitch>

*T:*+19184209001 | *F:*+19184209002 | *M:*+1918424WEST (9378)
*iNUM:*+883 5100 1420 9001 | *ISN:*410*543 | *Skype:*briankwest
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.freeswitch.org/pipermail/freeswitch-users/attachments/20151002/2fa9dbb7/attachment-0001.html 


Join us at ClueCon 2016 Aug 8-12, 2016
More information about the FreeSWITCH-users mailing list