Change in vdsm[master]: FIX Gal's comments

danken at redhat.com danken at redhat.com
Mon May 28 09:32:41 UTC 2012


Dan Kenigsberg has posted comments on this change.

Change subject: FIX Gal's comments
......................................................................


Patch Set 1: I would prefer that you didn't submit this

(1 inline comment)

please squash this patch into the former one. in this case, the separation into two patches does not help the review.

....................................................
File tests/capsTests.py
Line 62:         for res, s in map(None, expectedRes, sign):
zip() does it more transparently.

--
To view, visit http://gerrit.ovirt.org/4832
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5dd599982161e024bd85b50926bdbbe12d9545f3
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yeela Kaplan <ykaplan at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>


More information about the vdsm-patches mailing list