Code reviews in Launchpad and Bundle Buggy

John Arbash Meinel john at arbash-meinel.com
Mon Jun 2 18:19:31 BST 2008


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

Aaron Bentley wrote:
| 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

One other small thing would be nice. Being able to see the "threaded comments"
inline, rather than having to follow the link, and then come back to the same
page to go to the next item.

John
=:->
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkhEK6MACgkQJdeBCYSNAAMD+ACg14GcdiVDtDVNjs4CbF1CiWQy
NSIAoIxRGm+5yzTb/7qeZmBxUlJ3+YwP
=LBy4
-----END PGP SIGNATURE-----



More information about the bazaar mailing list