Sort issues by their age in the commit queue.

context.py

Index: context.py
diff --git a/context.py b/context.py
index 60ae8280fe13049c72c2d47b315eca4221e5bbb5..bb12881f3089e4b9131be54027928812ec064ba5 100644
--- a/context.py
+++ b/context.py
@@ -5,12 +5,14 @@
 
 class Context(object):
   """Class to hold context about a the current code review and checkout."""
-  def __init__(self, rietveld, checkout, status, server_hooks_missing=False):
+  def __init__(self, rietveld, checkout, status, user,
+               server_hooks_missing=False):
     """
     Args:
       rietveld: Instance of rietveld.Rietveld.
       checkout: Instance of checkout.SvnCheckout
       status: Instance of async_push.AsyncPush.
+      user: The --user option to commit_queue

[Paweł Hajdan Jr., 2014/01/29 02:38:24]

Oh, since this also has Rietveld, does using just rietveld.email work?

If it doesn't, how about changing the documentation of the user argument to say
"Rietveld user" instead?

[Sergey Berezin, 2014/01/29 17:41:43]

Actually, in a dry run, Rietveld receives an empty email (to signal "no
authentication"), while commit queue still has the specified (or default) user
name. We can discuss if this discrepancy should be eliminated later, but for now
I can't rely on rietveld.email.

On 2014/01/29 02:38:24, Paweł Hajdan Jr. wrote:

[Paweł Hajdan Jr., 2014/01/29 18:49:47]

Will it break in dry run? I think that unless it'd be broken it's fine - using
rietveld.email would be simpler it seems.

       server_hooks_missing: True if the project's SVN repository does not have
                             server-side hooks configured.
     """
@@ -18,3 +20,4 @@ class Context(object):
     self.checkout = checkout
     self.status = status
     self.server_hooks_missing = server_hooks_missing
+    self.user = user