<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sun, Jul 6, 2014 at 4:42 AM, Nick Lewycky <span dir="ltr"><<a href="mailto:nicholas@mxc.ca" target="_blank">nicholas@mxc.ca</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Chandler Carruth wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">
<br>
On Thu, Jul 3, 2014 at 11:00 PM, Nick Lewycky <<a href="mailto:nicholas@mxc.ca" target="_blank">nicholas@mxc.ca</a><br></div><div class="">
<mailto:<a href="mailto:nicholas@mxc.ca" target="_blank">nicholas@mxc.ca</a>>> wrote:<br>
<br>
    I don't like the lack attached patch files on the mailing list to do<br>
    a normal review.<br>
<br>
<br>
Wait what? The emails I get from phab *have* an attached patch file.<br>
That was a hard requirement when we first set up Phabricator.<br>
</div></blockquote>
<br>
Aaron nailed it. The initial emails come with attached patches. The problem is when people comment with the changes they made to the code, but there's no updated patch attached to that email. Aaron found examples so I'll defer to those. I can also keep an eye out for the next time it happens if you want.</blockquote>
<div><br></div><div>You should usually see 2 messages directly after each other - one with the patch, and one with the comment updates.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
<br>
Nick<br>
</font></span></blockquote></div><br></div></div>