launchpad-reviewers team mailing list archive

Thread
Date

Re: [Merge] lp:~rpadovani/launchpad/link-revision-merged-mp into lp:launchpad

To: Riccardo Padovani <riccardo@xxxxxxxxxxxxx>
From: Bayard 'kit' Randel <kit.randel@xxxxxxxxxxxxx>
Date: Mon, 13 Jul 2015 22:59:05 -0000
In-reply-to: <20150713222821.13847.64925.launchpad@ackee.canonical.com>
Reply-to: mp+264654@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing

Hi Riccardo, thanks for the patch!

This doesn't seem quite right however - currently this creates a link to code.launchpad.dev/project/branch/revno whereas I think we need (in the bazaar case), bazaar.launchpad.net/[project, ~user/project]/branch/revision/revno
-- 
https://code.launchpad.net/~rpadovani/launchpad/link-revision-merged-mp/+merge/264654
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.

References

[Merge] lp:~rpadovani/launchpad/link-revision-merged-mp into lp:launchpad
From: Riccardo Padovani, 2015-07-13