Nir Soffer has submitted this change and it was merged. Change subject: sp: Sort dict literal by key ......................................................................
sp: Sort dict literal by key With more than few items, it is easier to work with sorted items, and people are less likely to add duplicate keys at the end, as seen in commit f9cf58b7bced. Change-Id: I128ed804fcdbdf7b6b562f5cef662ae2ba2c9014 Signed-off-by: Nir Soffer <nsof...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/65052 Reviewed-by: Allon Mureinik <amure...@redhat.com> Reviewed-by: Freddy Rolland <froll...@redhat.com> Continuous-Integration: Jenkins CI Reviewed-by: Adam Litke <ali...@redhat.com> --- M vdsm/storage/sp.py 1 file changed, 6 insertions(+), 6 deletions(-) Approvals: Adam Litke: Looks good to me, approved Nir Soffer: Verified Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, but someone else must approve Freddy Rolland: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/65052 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I128ed804fcdbdf7b6b562f5cef662ae2ba2c9014 Gerrit-PatchSet: 11 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Adam Litke <ali...@redhat.com> Gerrit-Reviewer: Ala Hino <ah...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Freddy Rolland <froll...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> _______________________________________________ vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org