Merge lp://staging/~dtrg21/ubuntu-docviewer-app/add-readmes into lp://staging/ubuntu-docviewer-app
Proposed by
Aditya
Status: | Rejected |
---|---|
Rejected by: | Stefano Verzegnassi |
Proposed branch: | lp://staging/~dtrg21/ubuntu-docviewer-app/add-readmes |
Merge into: | lp://staging/ubuntu-docviewer-app |
To merge this branch: | bzr merge lp://staging/~dtrg21/ubuntu-docviewer-app/add-readmes |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Stefano Verzegnassi | Disapprove | ||
Andrew Hayzen (community) | Abstain | ||
Nicholas Skaggs | Pending | ||
Alan Pope πΊπ§π± π¦ | Pending | ||
Review via email: mp+281340@code.staging.launchpad.net |
Description of the change
Task from Google Code In. Added README's.
To post a comment you must log in.
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 docviewer- app/
$ cd ubuntu-
$ find | grep tst
$
0 - http:// bazaar. launchpad. net/~dtrg21/ ubuntu- docviewer- app/add- readmes/ revision/ 239