Code review comment for lp:~pwlars/checkbox/checkbox.webcamtest

Revision history for this message
Daniel Manrique (roadmr) wrote :

Hi Paul,

Thanks so much for your contribution to checkbox. I also offer you my sincerest apologies for the time it took us to review this submission. Unfortunately, since the time you sent this merge request, a suite of several camera-related tests were developed for other purposes and added to checkbox, and will be included in checkbox 0.12 to ship with Ubuntu 11.10 in a few months. Thus, since the functionality you suggest is already implemented in the development version, I will mark this branch as disapproved, just to signify that it will not be included in checkbox.

If you like, you can branch the latest checkbox source code and see if you can suggest some improvements to our tests. We are doing our best to review contributions in a timely manner.

Thanks again!

- Daniel

review: Disapprove

« Back to merge proposal