<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Also needs a note about the pull request name should include the jira that the pull request is for<div><br><div style=""><div>On Aug 29, 2014, at 11:27 AM, William King &lt;<a href="mailto:wking@freeswitch.org">wking@freeswitch.org</a>&gt; wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">Looks great. Not sure how best to display this information, but each<br>pull request tracks the branch from the forked repo. So if they update<br>the branch, the pull request is automatically updated with the new<br>commits. This means the person who is contributing code needs to have a<br>separate branch for each pull request, and if a developer asks for a<br>change, then the contributor needs to make the change in the branch, and<br>push the new change.<br><br>William King<br><a href="mailto:wking@freeswitch.org">wking@freeswitch.org</a><br>FreeSWITCH Solutions, LLC<br>PO BOX 2531<br>Brookfield, WI 53008-2531<br>Twitter: @FreeSWITCH , @quentusrex<br><a href="http://www.freeswitchbook.com/">http://www.freeswitchbook.com</a><br><a href="http://www.freeswitchcookbook.com/">http://www.freeswitchcookbook.com</a><br><br>T: +1.213.286.0416<br><a href="Skype:quentusrex">Skype:quentusrex</a><br><br>On 08/29/2014 07:43 AM, Ítalo Rossi wrote:<br><blockquote type="cite">Here is the first version of the tutorial:<br><br><a href="https://confluence.freeswitch.org/display/FREESWITCH/Pull+Requests">https://confluence.freeswitch.org/display/FREESWITCH/Pull+Requests</a><br><br>It's in Community -&gt; Contributing Code -&gt; Pull Requests.<br><br>English is not my first language, so maybe there are some gramatical<br>issues that need to be fixed #).<br><br><br>2014-08-28 23:35 GMT-03:00 Ítalo Rossi &lt;<a href="mailto:italorossib@gmail.com">italorossib@gmail.com</a><br>&lt;<a href="mailto:italorossib@gmail.com">mailto:italorossib@gmail.com</a>&gt;&gt;:<br><br>&nbsp;&nbsp;&nbsp;Hi Ken,<br><br>&nbsp;&nbsp;&nbsp;Sure, I can do this tomorrow.<br><br>&nbsp;&nbsp;&nbsp;I think it's a good idea to put some screenshots, it helps to<br>&nbsp;&nbsp;&nbsp;understand whole process.<br><br>&nbsp;&nbsp;&nbsp;Hey guys, can one of you help me out with a little process doc?<br><br>&nbsp;&nbsp;&nbsp;We are trying to encourage people to use Pull Requests for Patches<br>&nbsp;&nbsp;&nbsp;and there seems to be a little confusion on how to do this.<br><br>&nbsp;&nbsp;&nbsp;Can someone write up this process nicely and post it on Confluence?<br><br>&nbsp;&nbsp;&nbsp;Here’s the steps<br><br>&nbsp;&nbsp;&nbsp;Login in to Stash<br>&nbsp;&nbsp;&nbsp;Go to Repo you want to add a patch to<br>&nbsp;&nbsp;&nbsp;Click the Fork Button.<br>&nbsp;&nbsp;&nbsp;In the PopUp, name your fork (it will default to FreeSWITCH, but you<br>&nbsp;&nbsp;&nbsp;may want to name it something more descriptive. Example<br>&nbsp;&nbsp;&nbsp;freeswitch-FS-1234 &nbsp;&nbsp;where FS-1234 is the name of a jira you are<br>&nbsp;&nbsp;&nbsp;addressing)<br>&nbsp;&nbsp;&nbsp;This will bring you to your fork repo.<br>&nbsp;&nbsp;&nbsp;Go to the “Settings” link on the menu bar, go to “Repository” link<br>&nbsp;&nbsp;&nbsp;under permissions on the left verticle menu<br>&nbsp;&nbsp;&nbsp;Check the enable box under Public Access.<br>&nbsp;&nbsp;&nbsp;Click Save<br><br>&nbsp;&nbsp;&nbsp;Clone this repo, commit your patches, and push this repo back to<br>&nbsp;&nbsp;&nbsp;stash. (standard GIT clone, commit, push workflow)<br><br>&nbsp;&nbsp;&nbsp;Return to Stash, Go to the Main FreeSWITCH Repo, and click “Pull<br>&nbsp;&nbsp;&nbsp;Request” Button Upper right<br>&nbsp;&nbsp;&nbsp;In the box on upper left, select your fork repo and branch that<br>&nbsp;&nbsp;&nbsp;contains the new commits.<br>&nbsp;&nbsp;&nbsp;Make sure that the FreeSWITCH/freeswitch repo is selected in the<br>&nbsp;&nbsp;&nbsp;upper right box.<br><br>&nbsp;&nbsp;&nbsp;Edit the Title (it will default to the branch name)<br>&nbsp;&nbsp;&nbsp;Edit the Description (if there is a matching Jira Please Reference<br>&nbsp;&nbsp;&nbsp;the Jira Ticket ID at the top of the description. The description<br>&nbsp;&nbsp;&nbsp;will auto populate with commit messages from commits on the branch<br>&nbsp;&nbsp;&nbsp;not already in FreeSWITCH master repo)<br><br>&nbsp;&nbsp;&nbsp;Click Create Pull Request.<br><br>&nbsp;&nbsp;&nbsp;If you have not received feedback on your Pull Request by the<br>&nbsp;&nbsp;&nbsp;regular Thursday Bug Hunt/Patch Review conference call, Call in to<br>&nbsp;&nbsp;&nbsp;the bridge at 2PM US Central Time (1900 GMT) and ask to have your<br>&nbsp;&nbsp;&nbsp;patch reviewed.<br><br>&nbsp;&nbsp;&nbsp;Documenter, Please reference the Confluence page for how to join the<br>&nbsp;&nbsp;&nbsp;conf calls.<br><br><br>&nbsp;&nbsp;&nbsp;Thanks<br>&nbsp;&nbsp;&nbsp;K<br><br><br><br><br><br><br><br>&nbsp;&nbsp;&nbsp;--<span class="Apple-converted-space">&nbsp;</span><br>&nbsp;&nbsp;&nbsp;Ken<br>&nbsp;&nbsp;&nbsp;_http://www.FreeSWITCH.org<br>&nbsp;&nbsp;&nbsp;http://www.ClueCon.com<br>&nbsp;&nbsp;&nbsp;http://www.OSTAG.org<br>&nbsp;&nbsp;&nbsp;_<a href="http://irc.freenode.net/">irc.freenode.net</a><span class="Apple-converted-space">&nbsp;</span>&lt;<a href="http://irc.freenode.net/">http://irc.freenode.net</a>&gt; #freeswitch<br>&nbsp;&nbsp;&nbsp;Twitter: @FreeSWITCH<br><br><br>&nbsp;&nbsp;&nbsp;_______________________________________________<br>&nbsp;&nbsp;&nbsp;Freeswitch-docs mailing list<br>&nbsp;&nbsp;&nbsp;<a href="mailto:Freeswitch-docs@lists.freeswitch.org">Freeswitch-docs@lists.freeswitch.org</a><br>&nbsp;&nbsp;&nbsp;&lt;<a href="mailto:Freeswitch-docs@lists.freeswitch.org">mailto:Freeswitch-docs@lists.freeswitch.org</a>&gt;<br>&nbsp;&nbsp;&nbsp;<a href="http://lists.freeswitch.org/mailman/listinfo/freeswitch-docs">http://lists.freeswitch.org/mailman/listinfo/freeswitch-docs</a><br><br><br><br><br>--<span class="Apple-converted-space">&nbsp;</span><br>Ítalo Rossi<br><br><br>_______________________________________________<br>Freeswitch-docs mailing list<br><a href="mailto:Freeswitch-docs@lists.freeswitch.org">Freeswitch-docs@lists.freeswitch.org</a><br><a href="http://lists.freeswitch.org/mailman/listinfo/freeswitch-docs">http://lists.freeswitch.org/mailman/listinfo/freeswitch-docs</a><br><br></blockquote><br>_______________________________________________<br>Freeswitch-docs mailing list<br><a href="mailto:Freeswitch-docs@lists.freeswitch.org">Freeswitch-docs@lists.freeswitch.org</a><br><a href="http://lists.freeswitch.org/mailman/listinfo/freeswitch-docs">http://lists.freeswitch.org/mailman/listinfo/freeswitch-docs</a></div></blockquote></div><br></div></body></html>