<div dir="ltr"><br><br><div class="gmail_quote">On Tue, Mar 10, 2015 at 6:20 PM Matt Mackall <<a href="mailto:mpm@selenic.com">mpm@selenic.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Mon, 2015-03-09 at 18:26 -0400, Brett Cannon wrote:<br>
> # HG changeset patch<br>
> # User Brett Cannon <<a href="mailto:brett@python.org" target="_blank">brett@python.org</a>><br>
> # Date 1425939985 14400<br>
> #      Mon Mar 09 18:26:25 2015 -0400<br>
> # Node ID 4c0f97d377a0d193743f1b43481dad<u></u>efc1880cd0<br>
> # Parent  fe74d55995393f6a72abce49da8d43<u></u>e5fe37cdcd<br>
> hglib: Move string literals in the test code to util.b() (issue4520)<br>
<br>
Various things spotted by our automated checks (check-code and<br>
check-commit in the hg repo):<br>
<br>
- please don't uppercase "Move" in summary above<br>
<br>
> +        rev0, node0 = self.client.commit(b('first'), addremove=True,<br>
> +                                         date=now.isoformat(' ').encode('latin-1'))<br>
<br>
- this and a couple other lines are too long<br>
<br>
> -        print rev0, node0<br>
> +        print(rev0, node0)<br>
<br>
- not portable<br></blockquote><div><br></div><div>Not portable how? It works in every version of Python because Python 2 prints a tuple while Python 3 prints two space-separate values. Something will have to be chosen in this instance for final Python 3 support since it's a syntax error otherwise.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
- and now that my eyes are drawn to it, not related to $SUMMARY<br>
<br></blockquote><div><br></div><div>Yes, that was a oversight on my part. Teasing apart my work in another repo and reconstructing it is tough. =) But since this was the last major patch little slip-ups like this shouldn't happen again.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I've fixed these up, but you might want to acquaint yourself with these<br>
scripts.<br></blockquote><div><br></div><div>Sure thing. I'll run them from now on.</div><div><br></div><div>-Brett</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
--<br>
Mathematics is the supreme nostalgia of our time.<br>
<br>
<br>
</blockquote></div></div>