<meta http-equiv="Content-Type" content="text/html; charset=utf-8"><div><div dir="auto">We could just start with a short comment like “looks good” for now, and then revisit the question next software meeting when more people are familiar with what functionality is available. </div></div><div dir="auto"><br></div><div dir="auto">Cheers</div><div dir="auto">Sean</div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 29, 2020 at 8:46 AM Mark Ito <<a href="mailto:marki@jlab.org">marki@jlab.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Mark,<br>
<br>
That is a good point, well taken. Anyone who was at the meeting <br>
yesterday care to take this one?<br>
<br>
   -- Mark<br>
<br>
On 4/29/20 8:08 AM, Mark-Macrae Dalton wrote:<br>
> It might be useful to spell out more clearly what is expected from a review of a pull request, other than clicking the button.<br>
><br>
> Mark<br>
><br>
>> On Apr 28, 2020, at 8:06 PM, Mark Ito <<a href="mailto:marki@jlab.org" target="_blank">marki@jlab.org</a>> wrote:<br>
>><br>
>> Cool. Good to know.<br>
>><br>
>> On 4/28/20 7:59 PM, Justin Stevens wrote:<br>
>>> FYI, just successfully executed the review -> merge PR sequence.  There’s basically just one more button to push to “Approve” the review, and then you can merge as usual.<br>
>>> -Justin<br>
>>><br>
>>>> On Apr 28, 2020, at 5:04 PM, Mark Ito <<a href="mailto:marki@jlab.org" target="_blank">marki@jlab.org</a>> wrote:<br>
>>>><br>
>>>> Folks,<br>
>>>><br>
>>>> At the software meeting just now, we decided to require "reviews" of pull requests before they can be merged onto the master branch. For now this is only for halld_sim and halld_recon. The webpages for the individual pull requests now reflect this. The idea is to prevent inadvertent changes to the master branches.<br>
>>>><br>
>>>>    -- Mark<br>
>>>><br>
>>>><br>
>>>> _______________________________________________<br>
>>>> Halld-offline mailing list<br>
>>>> <a href="mailto:Halld-offline@jlab.org" target="_blank">Halld-offline@jlab.org</a><br>
>>>> <a href="https://mailman.jlab.org/mailman/listinfo/halld-offline" rel="noreferrer" target="_blank">https://mailman.jlab.org/mailman/listinfo/halld-offline</a><br>
>> _______________________________________________<br>
>> Halld-offline mailing list<br>
>> <a href="mailto:Halld-offline@jlab.org" target="_blank">Halld-offline@jlab.org</a><br>
>> <a href="https://mailman.jlab.org/mailman/listinfo/halld-offline" rel="noreferrer" target="_blank">https://mailman.jlab.org/mailman/listinfo/halld-offline</a><br>
_______________________________________________<br>
Halld-offline mailing list<br>
<a href="mailto:Halld-offline@jlab.org" target="_blank">Halld-offline@jlab.org</a><br>
<a href="https://mailman.jlab.org/mailman/listinfo/halld-offline" rel="noreferrer" target="_blank">https://mailman.jlab.org/mailman/listinfo/halld-offline</a></blockquote></div></div>