<div dir="ltr">I've sent another pull request. It would be great if you could review it and give me feedback and how I can make it better :)<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 8, 2016 at 11:42 AM, Richhiey Thomas <span dir="ltr"><<a href="mailto:richhiey.thomas@gmail.com" target="_blank">richhiey.thomas@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div>Hey James,<br><br></div>Thanks for the guidance. I have forked the current master branch on github and added the changes to the required files.<br></div>Then I made a branch named master on my local fork and created a pull request to the main repo.<br></div>Do let me know if I've gone wrong or if there is something I can add or something that I missed out on.<br><br></div>Thanks :)<br></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 8, 2016 at 10:09 AM, James Aylett <span dir="ltr"><<a href="mailto:james-xapian@tartarus.org" target="_blank">james-xapian@tartarus.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On Fri, Apr 08, 2016 at 09:57:16AM -0400, Richhiey Thomas wrote:<br>
<br>
> Sorry to take so much time on this. Was down with coursework because the<br>
> semester end is nearing.<br>
<br>
</span>Not a problem -- that sort of thing is affecting a lot of people at<br>
the moment!<br>
<span><br>
> I used the latest development version which is 1.3.5 for this patch.<br>
> I have implemented the $match function and it works fine when I tested it<br>
> with the omegascript templates provided in the development omega source<br>
> tarball (1.3.5)<br>
<br>
</span>Great. In future you should work from a git checkout rather than a<br>
tarball, because it makes it less likely you'll run into conflicts<br>
with subsequent work (or rather, if you do, you can resolve them more<br>
easily).<br>
<span><br>
> So how do I go ahead with creating a merge request for the same. Do<br>
> I push these changes to the master branch?  I went through the<br>
> Xapian developer guide but I am not sure about which branch these<br>
> changes should be pushed to.<br>
<br>
</span>So there's two sides to this. Firstly in your local repository. The<br>
developer guide has a 'helpful workflow' which discusses this. It was<br>
previously tucked away in another part of the document, so it was easy<br>
to miss; hopefully it's clearer and more obvious now. It says [1]:<br>
<br>
> Create a branch in your local git repository<br>
><br>
> ...<br>
><br>
> You can check create your new branch:<br>
><br>
> $ git checkout -b feature-x<br>
><br>
> The branch name doesn't really matter, but you'll probably find it<br>
> easiest to name it something related to the work that you're doing.<br>
<br>
Secondly, in your forked repository on github. It's easiest just to<br>
use the name branch name. Then you can open a pull request against the<br>
upstream master. If you're not familiar with github forks and pull<br>
requests, their documentation is the place to start [2].<br>
<br>
(In some ways it doesn't actually matter what branch you use in your<br>
repositories, however giving them a useful name makes it easier for<br>
everyone to see what's going on, particularly when -- as I am at the<br>
moment -- dealing with branches coming from a large number of<br>
different people!)<br>
<br>
J<br>
<br>
<br>
[1] <a href="https://xapian-developer-guide.readthedocs.org/en/latest/contributing/workflow.html#create-a-branch-in-your-local-git-repository" rel="noreferrer" target="_blank">https://xapian-developer-guide.readthedocs.org/en/latest/contributing/workflow.html#create-a-branch-in-your-local-git-repository</a><br>
<br>
[2] <a href="https://help.github.com/categories/collaborating-on-projects-using-issues-and-pull-requests/" rel="noreferrer" target="_blank">https://help.github.com/categories/collaborating-on-projects-using-issues-and-pull-requests/</a><br>
<div><div><br>
--<br>
  James Aylett, occasional trouble-maker<br>
  <a href="http://xapian.org" rel="noreferrer" target="_blank">xapian.org</a><br>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>