← Back to team overview

ubuntu-touch-coreapps-reviewers team mailing list archive

Thread
Date

Re: [Merge] lp:~ahayzen/music-app/add-manual-tests-001 into lp:music-app

To: mp+272050@xxxxxxxxxxxxxxxxxx
From: Nicholas Skaggs <nskaggs@xxxxxxxxxx>
Date: Tue, 29 Sep 2015 23:16:48 -0000
In-reply-to: <20150923010305.23192.23253.launchpad@ackee.canonical.com>
Reply-to: mp+272050@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing

Prefix the tests file names with 'music-'

You are missing a colon
s/_description/_description:/g

Still deciding on the provider / no provider bit.
-- 
https://code.launchpad.net/~ahayzen/music-app/add-manual-tests-001/+merge/272050
Your team Music App Developers is subscribed to branch lp:music-app.