ubuntu-touch-coreapps-reviewers team mailing list archive

Thread
Date

ubuntu-touch-coreapps-reviewers team
Mailing list archive
Message #05109

Re: [Merge] lp:~ahayzen/music-app/qml-unit-test-filenames into lp:music-app

To: mp+270512@xxxxxxxxxxxxxxxxxx
From: Victor Thompson <victor.thompson@xxxxxxxxx>
Date: Fri, 25 Sep 2015 01:02:00 -0000
In-reply-to: <20150909113036.630.93260.launchpad@ackee.canonical.com>
Reply-to: mp+270512@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing

Just briefly looking this over. I think we should also add a README.unittests with instructions on running the tests. Also, if we have a script to run on the device, would it also make sense to have a script that runs all the tests locally?
-- 
https://code.launchpad.net/~ahayzen/music-app/qml-unit-test-filenames/+merge/270512
Your team Music App Developers is subscribed to branch lp:music-app.

References

[Merge] lp:~ahayzen/music-app/qml-unit-test-filenames into lp:music-app
From: Andrew Hayzen, 2015-09-09