)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":20996,"name":"Avinash Raghavendra","email":"avinash.raghavendra@ericsson.com","username":"eragavi"},"change_message_id":"28e4e03f726044758758c6c37e3b7fcc43ba21d1","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix return value of _clone_image_volume"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Callers are looking for an object or None, this should not"},{"line_number":10,"context_line":"return false."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"pylint error:"},{"line_number":13,"context_line":"E:1248,30: Instance of \u0027bool\u0027 has no \u0027id\u0027 member (no-member)"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I6f2cefb3e62a04b55086f0429194b1b650ee69dc"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa0719c6_b3f953cb","line":13,"range":{"start_line":9,"start_character":0,"end_line":13,"end_character":60},"updated":"2016-03-21 15:59:40.000000000","message":"does this fix a bug? if so, please mention it here, if not","commit_id":"c7ea6bdf04299d2d7514aede44d2b2ee184856fc"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"581f5711ca2197274f7e923dd5d84b1daa8593b9","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix return value of _clone_image_volume"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Callers are looking for an object or None, this should not"},{"line_number":10,"context_line":"return false."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"pylint error:"},{"line_number":13,"context_line":"E:1248,30: Instance of \u0027bool\u0027 has no \u0027id\u0027 member (no-member)"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I6f2cefb3e62a04b55086f0429194b1b650ee69dc"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa0719c6_f3c8eb76","line":13,"range":{"start_line":9,"start_character":0,"end_line":13,"end_character":60},"in_reply_to":"fa0719c6_b3f953cb","updated":"2016-03-21 16:00:44.000000000","message":"It fixes a problem in the code.  It is not something that merits a bug report, though.","commit_id":"c7ea6bdf04299d2d7514aede44d2b2ee184856fc"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"44943708a2439c879a5900031dc9472a9098cd51","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix return value of _clone_image_volume"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Callers are looking for an object or None, this should not"},{"line_number":10,"context_line":"return false."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"pylint error:"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"dab17558_2344be19","line":9,"range":{"start_line":9,"start_character":34,"end_line":9,"end_character":41},"updated":"2016-05-13 10:00:14.000000000","message":"I kind of disagree... Callers are not looking for None, as code reads `if not image_volume:`. So it\u0027s looking for a non zero value as evaluated by __nonzero__ instance method. So we can return None, False, tuple(), set(), [], or anything else and it\u0027s all valid.\n\nBut I\u0027m OK changing this just to make pylint happy.","commit_id":"2a0712476a4278a376a58774807ca2b6da4389b9"}]}
