Launchpad itself

Code review comment for lp:~henninge/launchpad/on-edge

  1. on-edge
  2. Merge into devel
Revision history for this message
Jeroen T. Vermeulen (jtv) wrote :

Looks fine. A few nits: "res" is not a very good variable name, so use "match" instead. Double blank line above get_last_automatic_stable_merge_revno. That's about it.

review: Approve (code)

« Back to merge proposal