I'm up for a discussion next week, with pizza and beer, exact priorities of those three unstated.<br><br><div class="gmail_quote">On Fri, Oct 7, 2011 at 10:57 AM, Cameron Hummels <span dir="ltr"><<a href="mailto:chummels@astro.columbia.edu">chummels@astro.columbia.edu</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">I totally agree with everything Britton stated.  The key here is to make<br>
it easy for other developers to check/test your work, with the onus of<br>
responsibility placed on the pull request initiator.  I'd love to<br>
discuss this more next week.<br>
<br>
Cameron<br>
<div><div></div><div class="h5"><br>
On 10/07/2011 10:49 AM, Matthew Turk wrote:<br>
> Hi Britton and Sam,<br>
><br>
> I found the process to be really great.  Jeff has been suggesting for<br>
> a while that we identify different areas in the code, levels of<br>
> severity of changes, and have sets of eyes on changes.  Since a bunch<br>
> of us (although not all of us!) will be in NYC next week, maybe we<br>
> could take an hour or so to try to brainstorm and talk (over pizza,<br>
> drinks, etc?) about pull requests and how we can and should use them?<br>
> It'd be good to walk the line Britton identifies, since we're all busy<br>
> people with lots of work to do, but we also don't want to introduce<br>
> regressions.  Guidelines would help ensure that burden of testing<br>
> changes isn't too high on the wrong people.<br>
><br>
> -Matt<br>
><br>
> On Fri, Oct 7, 2011 at 10:21 AM, Britton Smith <<a href="mailto:brittonsmith@gmail.com">brittonsmith@gmail.com</a>> wrote:<br>
>> I think this is a great model for development.  When it comes to submitting<br>
>> large new features or significant invasive changes, the onus must be on the<br>
>> original developer to make inspection and verification of incoming changes<br>
>> as easy as possible for reviewers.  In my opinion, this means pull requests<br>
>> of this nature should be accompanied by full explanation of what the code is<br>
>> doing, how it works, and how a reviewer should know whether it's working or<br>
>> not, including testing scripts.  Otherwise, it's just asking busy people to<br>
>> do more work.<br>
>><br>
>> For small bug fixes, I think it's enough to submit the pull request and have<br>
>> a few extra pairs of eyes look it over.  For bigger things, I think this is<br>
>> something to keep in mind.<br>
>><br>
>> Britton<br>
>><br>
>> On Thu, Oct 6, 2011 at 4:12 PM, Sam Skillman <<a href="mailto:samskillman@gmail.com">samskillman@gmail.com</a>> wrote:<br>
>>> Hi all,<br>
>>> I just wanted to relate the experience I just had with Matt's pull request<br>
>>> on the isocontour flux calculations.  If you already regard pull requests as<br>
>>> being awesome, you can probably stop reading.<br>
>>> The situation:<br>
>>> Matt had a fairly large set of changes that added new functionality to yt,<br>
>>> and wanted another pair of eyes before pulling it into the main trunk.<br>
>>> The tool: bitbucket pull requests<br>
>>> Matt developed the changes in his fork of the main yt branch, then<br>
>>> requested that they be pulled into the main repo (even though he had the<br>
>>> ability to push directly).<br>
>>> The benefit:<br>
>>> This allowed me to pull down his changes, test them, iterate back and<br>
>>> forth with him, and make comments that now forever live in the "Accepted<br>
>>> pull requests" part of the repo so that anyone can see why that was changed.<br>
>>>  When it was set to go, I just clicked "Accept" and the changes were all<br>
>>> merged in without incident.<br>
>>> Anyways, very effortless way to handle changes that are more than just one<br>
>>> liners and need a collaborative effort before going into the main branch.<br>
>>> Sam<br>
>>><br>
>>><br>
>>> _______________________________________________<br>
>>> Yt-dev mailing list<br>
>>> <a href="mailto:Yt-dev@lists.spacepope.org">Yt-dev@lists.spacepope.org</a><br>
>>> <a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" target="_blank">http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org</a><br>
>>><br>
>><br>
>> _______________________________________________<br>
>> Yt-dev mailing list<br>
>> <a href="mailto:Yt-dev@lists.spacepope.org">Yt-dev@lists.spacepope.org</a><br>
>> <a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" target="_blank">http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org</a><br>
>><br>
>><br>
> _______________________________________________<br>
> Yt-dev mailing list<br>
> <a href="mailto:Yt-dev@lists.spacepope.org">Yt-dev@lists.spacepope.org</a><br>
> <a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" target="_blank">http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org</a><br>
><br>
<br>
</div></div><font color="#888888">--<br>
Cameron Hummels<br>
PhD Candidate, Astronomy Department of Columbia University<br>
Public Outreach Director, Astronomy Department of Columbia University<br>
NASA IYA New York State Student Ambassador<br>
<a href="http://outreach.astro.columbia.edu" target="_blank">http://outreach.astro.columbia.edu</a><br>
PGP: 0x06F886E3<br>
</font><div><div></div><div class="h5"><br>
_______________________________________________<br>
Yt-dev mailing list<br>
<a href="mailto:Yt-dev@lists.spacepope.org">Yt-dev@lists.spacepope.org</a><br>
<a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" target="_blank">http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org</a><br>
</div></div></blockquote></div><br>