Code review comment for lp:~rodrigo-moya/ubuntuone-client/nautilus-refactor-1

Revision history for this message
Natalia Bidart (nataliabidart) wrote :

I ran the test though the Python suite won't cover changes in this branch. I performed some manual testing (start synching a folder, stop synching, publish file). No crashes!
The overview of the code looks fine, though I didn't follow it on detail. I still insist we need an automated test suite for all the C code.

review: Approve

« Back to merge proposal