Code review comment for lp:~wgrant/launchpad/flatten-bfj-3-query

Revision history for this message
Steve Kowalik (stevenk) wrote :

799 + ).group_by(
800 + BinaryPackageBuild._new_status
801 + ).order_by(BinaryPackageBuild._new_status)

Perhaps this could be re-flowed?

1011 + find_spec = (BinaryPackageBuild,)

Why? Surely this can just be inlined into the .find() call.

review: Approve (code)

« Back to merge proposal