ubuntu-touch-coreapps-reviewers team mailing list archive

Thread
Date

Re: [Merge] lp:~dtrg21/ubuntu-docviewer-app/add-readmes into lp:ubuntu-docviewer-app

To: Aditya <dtrg21@xxxxxxxxx>
From: Andrew Hayzen <ahayzen@xxxxxxxxx>
Date: Mon, 28 Dec 2015 14:38:23 -0000
In-reply-to: <20151224094837.32070.25444.launchpad@ackee.canonical.com>
Reply-to: mp+281340@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Abstain

Strange how there is no diff, that can be seen here [0]. Also I'm not that the unit test readme is required as I cannot spot any unit test files in the repo?

I've added the docviewer developers as a reviewer to this MP, so they can decide if it is required as I cannot spot any unittest files.

$ bzr branch lp:ubuntu-docviewer-app
$ cd ubuntu-docviewer-app/
$ find | grep tst
$


0 - http://bazaar.launchpad.net/~dtrg21/ubuntu-docviewer-app/add-readmes/revision/239
-- 
https://code.launchpad.net/~dtrg21/ubuntu-docviewer-app/add-readmes/+merge/281340
Your team Ubuntu Document Viewer Developers is requested to review the proposed merge of lp:~dtrg21/ubuntu-docviewer-app/add-readmes into lp:ubuntu-docviewer-app.