Code reviews in Launchpad and Bundle Buggy

[Aaron Bentley]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Nicholas Allen wrote:
> That's great! I wish the merge review could show side by side diff and
> more context than a few lines (a bit like reviewboard does). Is that a
> planned feature?

Showing a diff is definitely planned.  I'm not sure of the details.

> Also if someone submits a merge request then usually it receives
> comments and the author has to change things - in this case it would be
> nice to see the diffs between the 2 submitted versions so you see only
> the changes made to the patch since the last request. That way you can
> easily see if the issues raised have been addressed...

Generating such diffs is quite hard if the submitter has done any merges
in addition to the changes.

Aaron
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFIRCoN0F+nu1YWqI0RAqshAJ9I69UT3vZOI5j2vfum6V0sV9CWgQCgidqT
mdxDP8Ruty5e3/xLA1PG0LY=
=jQiV
-----END PGP SIGNATURE-----