Code review comment for lp:~mhr3/libunity/fix-sources-update

Revision history for this message
Mikkel Kamstrup Erlandsen (kamstrup) wrote :

Looking correct.

I was pondering for quite a while if we needed tests for this. I came to the conclusion that we didn't because upon inspecting the code the only line that changes the logic is an extra or-clause, which can only lead us to updating the scopes more often. Not to, fx, not updating them.

review: Approve

« Back to merge proposal