Code review comment for lp:~ahayzen/music-app/ap-helper-refactor-004

Revision history for this message
Victor Thompson (vthompson) wrote :

I think having this "virtual model" for metadata currently hurts the readability of the tests that use it, but I know having it will prove useful in the future. Also, the restructuring of some of the various loops looks good. Approved, but waiting for a secondary review by balloons.

review: Approve

« Back to merge proposal